Skip to content
This repository has been archived by the owner on Jun 3, 2019. It is now read-only.

Release: Version 14 #524

Open
1 of 4 tasks
strues opened this issue Oct 11, 2017 · 12 comments
Open
1 of 4 tasks

Release: Version 14 #524

strues opened this issue Oct 11, 2017 · 12 comments

Comments

@strues
Copy link
Collaborator

strues commented Oct 11, 2017

I haven't seen any check lists going around or a discussion on releasing the next branch so here we are.

  • Merge master bug fix pending PRs
  • Merge master changed into Next
  • Test Next
  • RenderToNodeStream?

What else am I forgetting? We should get this up to date and out since the release of React 16 seems to be smooth.

@oyeanuj
Copy link
Contributor

oyeanuj commented Oct 11, 2017

Yes! We should also go through the ideas/suggestions for 'next' ticket and see which of those can be included: #428

@ctrlplusb
Copy link
Owner

Hey all! Thanks for the activity here.

Please would all contributors get active on the gitter channel. Thanks 👍

I'd like us to start using a git rebase strategy for next and all pull requests in order to keep the merge noise down. If you are unfamiliar with git rebasing then please let me know and I would be happy to take you through it. 👍

Please keep all changes targetting next rather than master. I know we have already let a few slip into master but moving forward it would be best to use next to stage up some changes before me merge in.

I'll review #428 and give some more of my ideas.

Also, has anyone tried react-loadable against this project? James has added some cool APIs such as the preloadAll. It may be worth trying a branch out and seeing how it flies.

I'd also like to start killing the feature branches. Really keep them down to 2 or 3 at an absolute maximum. I know we had an issue for this already. I'll revisit it soon.

@codepunkt
Copy link
Collaborator

How does this work with css-in-js libraries, e.g. styled-components? Will this be a problem for branches?

@sergiokopplin
Copy link
Collaborator

@ctrlplusb which branches you think will be good to maintain?

@sergiokopplin
Copy link
Collaborator

@codepunkt you mean 'RenderToNodeStream' with css-in-js?

@codepunkt
Copy link
Collaborator

@sergiokopplin Yeah. Just noticed i've commented into the wrong issue. Sorry for that!

@oyeanuj
Copy link
Contributor

oyeanuj commented Oct 29, 2017

Another topic for v14:

// For our client bundles we transpile all the latest ratified
// ES201X code into ES5, safe for browsers. We exclude module
// transilation as webpack takes care of this for us, doing
// tree shaking in the process.
ifClient(['env', { es2015: { modules: false } }]),

I noticed in the configFactory here that we create client bundles in ES5. I was researching a little bit more and it seems like if we set up two bundles - one for modern browsers and one for non-modern browsers, we can make a significant dent on bundle sizes.

Still new to this topic but most of my understanding comes from this article: https://philipwalton.com/articles/deploying-es2015-code-in-production-today/

#514 and #531 seem to also be related to this.

@birkir
Copy link
Collaborator

birkir commented Nov 2, 2017

I've been busy in React Native world, please let me know if you guys need help.
It's my favourite project so I'm looking forward to see in what direction we're going.

@saniko
Copy link

saniko commented Mar 4, 2018

Hello,
Any updates on next version?

Best

@birkir
Copy link
Collaborator

birkir commented Mar 13, 2018

next now has both webpack 4 and react 16

master...next

Question @ctrlplusb , are we using prettier-eslint or no? There seems to be an inconsistency between branches regarding prettier. All pull-requests are now including prettier junk, making it very hard to glance PR's.

Thanks

@ctrlplusb
Copy link
Owner

I am keen on vanilla prettier (i.e. without the prettier-eslint). We should also use eslint-preset-prettier alongside eslint-preset-airbnb in order to disable all the style rules. 👍

@oyeanuj
Copy link
Contributor

oyeanuj commented Mar 17, 2018

@birkir Thoughts on #524 (comment) with webpack 4?

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

No branches or pull requests

7 participants