Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

An in-range update of redux is breaking the build 🚨 #74

Open
greenkeeper bot opened this issue Jun 17, 2017 · 3 comments
Open

An in-range update of redux is breaking the build 🚨 #74

greenkeeper bot opened this issue Jun 17, 2017 · 3 comments

Comments

@greenkeeper
Copy link

greenkeeper bot commented Jun 17, 2017

Version 3.7.0 of redux just got published.

Branch Build failing 🚨
Dependency redux
Current Version 3.6.0
Type devDependency

This version is covered by your current version range and after updating it in your project the build failed.

As redux is “only” a devDependency of this project it might not break production or downstream projects, but “only” your build or test tools – preventing new deploys or publishes.

I recommend you give this issue a high priority. I’m sure you can resolve this 💪

Status Details
  • ci/circleci Your tests failed on CircleCI Details

Release Notes v3.7.0

Another long break!

Oh, hey! I didn't see you sitting there. You look bored. How about a Redux release to spice things up?

Not a huge set of changes to report here. The biggest change, and the reason for the minor bump, is the UMD build is now done via Rollup. One big advantage is more readable code in the bundle. Rollup does "scope hoisting", which is a fancy term for putting every module at the top level of the file. Other than a surrounding IIFE, all of the code in Redux all lives together. You can compare the two here:

Rollup UMD build
vs
Webpack UMD build

There is also a cost savings of 30,811 vs 26,880 bytes, and 6,999 vs 5,995 bytes minified. Redux is already a small library, and this helps shave some extra bytes for our UMD users.

One thing to note is that Webpack has introduced it's own scope hoisting feature in 3.0 beta. So, this isn't intended as an indictment of Webpack. You should continue to use it in your own apps. The adage of "Webpack is for apps, Rollup is for libraries" definitely holds true. It still has a superior developer experience with hot module reloading and webpack-dev-server. But use whatever makes sense for your project, not just whatever we use. 😄

We're also looking at applying this to the NPM bundle. The main motivation is again more readable code in your bundles. Instead of transpilation oddities from Babel, you will end up with a single clean file, which should be easier to read through and debug. It's currently scheduled for the big, mythical 4.0 release and you can follow along in #2358

Changes

Commits

The new version differs by 309 commits.

  • 2d229f0 3.7.0
  • f3bba96 Removing the browser field for now
  • f4d9e55 Upgrade all example deps, including the universal example to Webpack 2
  • 2bc8f84 Upgrade some other deps
  • 070b838 Upgrades and remove check-es3-syntax since Rollup breaks it always
  • 1094724 Fix a typo in the Immutable.js docs (#2453)
  • bf3a557 Fix redux-mock-store libdef for todos-flow (#2430)
  • bf2b9b5 give redux-subscriber some attention in StoreSetup.md (#2433)
  • 8b3e0cb syncing docs with recent PR (2431) (#2432)
  • e238a19 'todo' const removal from reducers/todos.js (#2431)
  • 0babfab Merge pull request #2429 from alexbaumgertner/bugfix/doc-link
  • ae563c3 Fix link to ImmutableData article
  • 383215a Update redux.js (#2428)
  • c9ad6a4 Add 100% test coverage for todos-flow example (#2413)
  • ea16d76 Fix eslint warnings, update react-scripts (#2420)

There are 250 commits in total.

See the full diff

Not sure how things should work exactly?

There is a collection of frequently asked questions and of course you may always ask my humans.


Your Greenkeeper Bot 🌴

@greenkeeper
Copy link
Author

greenkeeper bot commented Jun 17, 2017

After pinning to 3.6.0 your tests are still failing. The reported issue might not affect your project. These imprecisions are caused by inconsistent test results.

@greenkeeper
Copy link
Author

greenkeeper bot commented Jun 26, 2017

Version 3.7.1 just got published.

Your tests are passing again with this version. Explicitly upgrade to this version 🚀

Release Notes v3.7.1

This reverts the console.error on bindActionCreators() coming from #2279.

While well-intentioned, when star importing all exports from a module (import * as actions from './actions'), transpilation by Babel defaults to adding a default and __esModule property to the import, which are not functions. While it can be disabled, this isn't common to do and leads to a lot of confusion. So, we're reverting the change.

Thanks for the feedback from everyone and the civility and healthy discourse on the issue!

Commits

The new version differs by 10 commits.

  • 6eed34d Switch prepublish to prepare script for npm 5 warning
  • 7fbddd6 3.7.1
  • 2bdcf66 Revert "Add warning when bindActionCreators encounters non-function property (#2279)" (#2473)
  • 6c7fe8e Merge pull request #2472 from jimmyhmiller/fix-dot
  • cb493d5 Merge pull request #2471 from BeLi4L/patch-1
  • ec5b9c7 removed extra dot
  • c5bfb97 docs: fix wrong parameters
  • 9e5feb2 Moved propTypes checking, fixed undefined error (#2464)
  • 689c800 Merge pull request #2463 from taehwanno/docs-change-array-reduce
  • 4155297 Change Array.reduce to Array.prototype.reduce

See the full diff

@greenkeeper
Copy link
Author

greenkeeper bot commented Jul 13, 2017

Version 3.7.2 just got published.

Your tests are passing again with this version. Explicitly upgrade to this version 🚀

Release Notes v3.7.2

Changes

Commits

The new version differs by 19 commits.

  • 8f60ba3 3.7.2
  • f4c908b Add new discussion links to Performance (#2500)
  • b8cf8aa writing tests for middleware example changed (#2496)
  • 7c68f34 Add new links to Code Structure (#2494)
  • d019f11 Fix a typo (#2497)
  • e9067e6 Add link to Full Stack React article
  • c1953b0 Add sandboxes to the examples in the docs (#2487)
  • 6ca30fb Add Sandboxes for almost all examples (#2486)
  • 0de7e5b Fixed the definition of 'Reducer' for TypeScript 2.4. (#2467)
  • 6eed34d Switch prepublish to prepare script for npm 5 warning
  • 7fbddd6 3.7.1
  • 2bdcf66 Revert "Add warning when bindActionCreators encounters non-function property (#2279)" (#2473)
  • 6c7fe8e Merge pull request #2472 from jimmyhmiller/fix-dot
  • cb493d5 Merge pull request #2471 from BeLi4L/patch-1
  • ec5b9c7 removed extra dot

There are 19 commits in total.

See the full diff

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

No branches or pull requests

0 participants