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

Upgrades docs pages to use React 15.3.2 #584

Merged
merged 3 commits into from
Oct 19, 2016
Merged

Conversation

mute
Copy link
Contributor

@mute mute commented Oct 19, 2016

PR Checklist

  • Manually tested across supported browsers
    • Chrome
    • Firefox
    • Safari
    • IE11 (Win 7)
    • Edge (Win 10)
  • Unit tests written (common at minimum)
  • PR has one of the semver- labels
  • Two core team engineer approvals
  • [ ] One core team UX approval

Removes React < 15 from peerDeps

Fixes #582

Docs here: http://docspot.devnxs.net/projects/lucid/582-upgrade-to-react-15/

Removes React < 15 from peerDeps
@mute mute self-assigned this Oct 19, 2016
@@ -230,7 +234,7 @@ describe('component-types', () => {
);
});

it('should find all elements of the same type from props and children', () => {
it('should find all elements of the same type from props and children', (done) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

i don't get this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, I'm not too sure why it was failing after the upgrade, either.

This fixed it though.

@mute mute merged commit 3ac3854 into master Oct 19, 2016
@mute mute deleted the 582-upgrade-to-react-15 branch October 19, 2016 23:29
@mute mute removed the in progress label Oct 19, 2016
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