Skip to content

Conversation

taion
Copy link
Contributor

@taion taion commented Jul 1, 2016

Among other things, this makes our CI run against the new React release.

"__DEV__": true
},
"rules": {
"react/jsx-uses-react": 1,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are in the recommended config

@taion
Copy link
Contributor Author

taion commented Jul 1, 2016

Are we okay to merge this?

@timdorr timdorr merged commit 9c955c5 into master Jul 2, 2016
@timdorr timdorr deleted the chore branch July 2, 2016 02:19
@taion
Copy link
Contributor Author

taion commented Jul 2, 2016

Okay so I started changing the ESLint config but then I realized that we have both a master and next branch going, and updating code style is a recipe for making ALL THE MERGE CONFLICTS between the two branches. So I stopped. It'll be better to wait until we've gotten back down to one development branch before making further code style changes.

We did that once for React-Bootstrap (update code style while running a next branch for prereleases) and resolving the merge conflicts there was just about the most painful experience I've ever had in maintaining an open-source library.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
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.

2 participants