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

15.5 bump proptypes version #9403

Closed

Conversation

flarnie
Copy link
Contributor

@flarnie flarnie commented Apr 11, 2017

To fix a breaking change which was not supposed to be included yet, we
reverted the change in prop-types v15.5.7 and are now pulling that into
React.

Related to fixing facebook/prop-types#17

To fix a breaking change which was not supposed to be included yet, we
reverted the change in prop-types v15.5.7 and are now pulling that into
React.

Related to fixing facebook/prop-types#17
@flarnie
Copy link
Contributor Author

flarnie commented Apr 11, 2017

Probably CI will fail until we actually release the updated version of the 'prop-types' package.

@bvaughn
Copy link
Contributor

bvaughn commented Apr 11, 2017

This change looks good, but maybe wait to merge until we've actually released prop-types 15.5.7?

Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

I don’t think this is right—it changes the version in root package.json which is used for devDependencies (e.g. tests—not sure why prop-types was there), but we should change packages/react/package.json instead.

@gaearon
Copy link
Collaborator

gaearon commented Apr 11, 2017

^^ I'll probably do this manually because our branches are a little messed up anyway and it's easier to do by hand during the release

@gaearon
Copy link
Collaborator

gaearon commented Apr 11, 2017

Closing as I did this manually.

@gaearon gaearon closed this Apr 11, 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