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

Flatten everything #11304

Merged
merged 3 commits into from
Oct 20, 2017
Merged

Flatten everything #11304

merged 3 commits into from
Oct 20, 2017

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Oct 20, 2017

Makes so that we're at most one level deep (excluding src) into each package.
Added new events pseudo-package that technically works like shared and was extracted from it.

  |-events
  |-react
  |  |-npm
  |  |-src
  |-react-art
  |  |-npm
  |  |-src
  |-react-cs-renderer
  |  |-src
  |-react-dom
  |  |-npm
  |  |-src
  |  |  |-client
  |  |  |-events
  |  |  |-server
  |  |  |-shared
  |  |  |-test-utils
  |  |  |-unstable-native-dependencies
  |-react-native-renderer
  |  |-src
  |-react-noop-renderer
  |  |-npm
  |  |-src
  |-react-reconciler
  |  |-npm
  |  |-src
  |-react-rt-renderer
  |  |-src
  |-react-test-renderer
  |  |-npm
  |  |-src
  |-shared

This makes #11303 an easier pill to swallow.

Copy link
Collaborator

@sebmarkbage sebmarkbage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hehe. Flatlandia. Classic Dan.

@gaearon gaearon merged commit e779c39 into facebook:master Oct 20, 2017
@jquense
Copy link
Contributor

jquense commented Oct 24, 2017

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.

4 participants