Skip to content

[WIP] no more vars on packages#11736

Closed
raphamorim wants to merge 13 commits intofacebook:masterfrom
raphamorim:no-more-vars-on-packages
Closed

[WIP] no more vars on packages#11736
raphamorim wants to merge 13 commits intofacebook:masterfrom
raphamorim:no-more-vars-on-packages

Conversation

@raphamorim
Copy link
Copy Markdown
Contributor

@raphamorim raphamorim commented Dec 1, 2017

Removing remaining vars
REF: #11732

Removed entirely:

  • events
  • react
  • react-art
  • react-call-return
  • react-cs-renderer
  • react-dom
  • react-native-renderer
  • react-noop-renderer
  • react-reconciler
  • react-rt-renderer
  • react-test-renderer
  • shared

@raphamorim raphamorim force-pushed the no-more-vars-on-packages branch from aaffa04 to df25386 Compare December 1, 2017 02:41
@raphamorim raphamorim force-pushed the no-more-vars-on-packages branch from 2f1557b to f9ca024 Compare December 1, 2017 02:54
@gaearon
Copy link
Copy Markdown
Collaborator

gaearon commented Dec 1, 2017

I've already prepared a change to some files in ReactDOM that I plan to land tomorrow so this might need to wait and be fixed up later.

@raphamorim
Copy link
Copy Markdown
Contributor Author

raphamorim commented Dec 1, 2017

Ok, thanks for the advice.
I'll wait the these changes to finish React-DOM (by now still missing: events, client folders).

Tomorrow I rebase from your changes and finish it.

@raphamorim raphamorim force-pushed the no-more-vars-on-packages branch from d4da334 to 7c0d68e Compare December 2, 2017 21:55
@raphamorim raphamorim mentioned this pull request Dec 2, 2017
@raphamorim
Copy link
Copy Markdown
Contributor Author

raphamorim commented Dec 2, 2017

@gaearon I underestimated this, I made a new PR with commits based on the changed folders to facilitate the review.

Finished in #11752

@raphamorim raphamorim closed this Dec 2, 2017
@raphamorim raphamorim deleted the no-more-vars-on-packages branch December 2, 2017 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants