Skip to content
This repository has been archived by the owner on Dec 24, 2024. It is now read-only.

fix: small fixes to the types and implement cjs2esm #70

Merged
merged 6 commits into from
Aug 10, 2020
Merged

Conversation

homer0
Copy link
Owner

@homer0 homer0 commented Aug 10, 2020

What does this PR do?

The two highlights of this PR are:

  1. I created specific types for the all providers-things as it's a better experience for the projects implementing the providers from here.
  2. I implemented cjs2esm for the ESM version generation; it's a wrapper on top of the script I already had here, but it also takes care of adding the extensions on the import statements.

How should it be tested manually?

npm test
# or
yarn test

homer0 added 6 commits August 7, 2020 16:41
I updated the dependencies and I re installed Jimple as a development dependency, just for the type.
This is actually a 'docs:', but I need to trigger a release.
I changed the prepare script to prepublishOnly and instead of doing the transformation manually, I
installed cjs2esm: it does the same, but it also fixes the issue with the missing extensions.
@homer0 homer0 changed the title Small fixes to the types and implement cjs2esm fix: small fixes to the types and implement cjs2esm Aug 10, 2020
@homer0 homer0 merged commit 79137a4 into master Aug 10, 2020
@homer0 homer0 deleted the homer0_fixes branch August 10, 2020 07:34
github-actions bot pushed a commit that referenced this pull request Aug 10, 2020
## [4.0.1](4.0.0...4.0.1) (2020-08-10)

### Bug Fixes

* small fixes to the types and implement cjs2esm ([79137a4](79137a4)), closes [#70](#70)
* use cjs2esm for the ESM version ([d65e109](d65e109))
* use proper types for providers ([ab5691e](ab5691e))
@github-actions
Copy link

🎉 This PR is included in version 4.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant