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

upgrade dependencies. #561

Closed
wants to merge 5 commits into from
Closed

upgrade dependencies. #561

wants to merge 5 commits into from

Conversation

l-7-l
Copy link

@l-7-l l-7-l commented Mar 5, 2018

This PR mainly upgrades webpack and some dependencies

@ctrlplusb @hguillermo Please give me some advice.

@l-7-l
Copy link
Author

l-7-l commented Mar 7, 2018

@hguillermo
Copy link

@chenyang-biu close to finish your PR? I would like to try this boilerplate with react 16. If I can help let me know.

@l-7-l l-7-l changed the title WIP: upgrade dependencies. upgrade dependencies. Mar 10, 2018
@l-7-l
Copy link
Author

l-7-l commented Mar 10, 2018

@hguillermo I'm sorry, I've been looking for work these days....

@l-7-l
Copy link
Author

l-7-l commented Mar 10, 2018

please enlighten.

@l-7-l l-7-l closed this Mar 12, 2018
@l-7-l l-7-l reopened this Mar 12, 2018
@sergiokopplin sergiokopplin mentioned this pull request Mar 13, 2018
@birkir
Copy link
Collaborator

birkir commented Mar 13, 2018

Thanks for the PR but now the next branch has everything (and more).

@birkir birkir closed this Mar 13, 2018
@sergiokopplin
Copy link
Collaborator

👏

@hguillermo
Copy link

Thanks @birkir. Are you planning to merged next to master soonish?

@birkir
Copy link
Collaborator

birkir commented Mar 13, 2018

We are waiting on one dependency to be released, react-tree-walker 👍

@hguillermo
Copy link

Perfect. I guess I will start from next and update it later.

@unleashit
Copy link

Just wondering... are the new updates to react-tree-walker going to support renderToNodeStream, or will the app still have to use renderToString first? Seems like one of the primary benefits to R16 is the ability to stream server rendering, which appreas to not be happening right now with this repo except at the end as a string.

@hguillermo
Copy link

@birkir Do you know when are you merging next to master? What are the pending things if I use next as a base for my next project? Thanks

@birkir
Copy link
Collaborator

birkir commented Apr 4, 2018

I do not.
My buddy, the project owner, @ctrlplusb is the man to ask! 👍

@hguillermo
Copy link

Thanks @birkir.

@ctrlplusb: when do you think you will merge next to master? is it something pending?

@l-7-l l-7-l deleted the upgradeDependencies branch April 7, 2018 01:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants