Migrate to mjs #276
Merged
Migrate to mjs #276
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I've labelled this a breaking change, because the primary file names have changed. For users who would have had something like Generally we don't consider changes in file names breaking, unless those names were documented. But I think for the main files a major version bump might be wise. |
Codecov Report
@@ Coverage Diff @@
## master #276 +/- ##
======================================
Coverage 100% 100%
======================================
Files 48 48
Lines 978 984 +6
Branches 0 210 +210
======================================
+ Hits 978 984 +6
Continue to review full report at Codecov.
|
Closed
You're unlikely to run out of major version numbers. 😄 |
Avaq
added a commit
that referenced
this pull request
Aug 30, 2018
Breaking changes - #276 The "module" package file has been renamed from index.mjs.js to index.mjs. - #277 It is no longer possible to use Future.ap, Future.map, Future.bimap, Future.chain, or Future.alt on non-Fantasy Land types. If you were using any of these functions from Fluture to treat JavaScript's native types as Fantasy Land algebraic types, you should migrate those call-sites to use the Ramda or Sanctuary exported dispatchers instead. - #281 Future.finally no longer runs the cleanup Future when it is cancelled. If you rely on this behaviour, I urge you to rewrite the relevant code with Future.hook (even if you don't upgrade Fluture). New features - #276 The Fluture source is now loadable by Node's experimental module loader, and the esm loader (https://github.com/standard-things/esm). - #280 The Future.of function now has an alias 'resolve', which is exported statically as well as exposed as a property on the Future constructor. Bug fixes and improvements - #277 Fluture no longer depends on sanctuary-type-classes, reducing bundle size. - #277 Performance of Fantasy Land dispatchers greatly improved. - #281 The undesired consequences of the cancellation-related behaviour in Fluture.finally are gone now that the behaviour has been removed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is (kind-of) the long-awaited followup of 96e1b95!
This change allows Node 9+ and
esmusers toimport {named} from 'fluture'. However, unlike what I said in 96e1b95, I will continue to include a.jsbuild in the published package until all major tooling has caught up with the new.mjsloader. The purpose of this change then becomes solely to support esm properly - something I thought was already the case after 96e1b95..jsfile extensions renamed to.mjs.mjsfiles, nor can Istanbul. So the unit tests will now be run usingesmwith the approach suggested in Issue upgrading to 3.0.57 - "Must use import to load ES module" standard-things/esm#498 (comment).Closes #274