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

Locking eslint-plugin-react to 2.5.0 #833

Closed
wants to merge 1 commit into from

Conversation

mtscout6
Copy link
Member

Due to breaking change in the 2.5.1 release.

See: jsx-eslint/eslint-plugin-react#110

@taion
Copy link
Member

taion commented Jun 14, 2015

Needs to be "2.5.0" rather than "^2.5.0". LGTM otherwise.

Due to breaking change in the 2.5.1 release.

See: jsx-eslint/eslint-plugin-react#110
@mtscout6
Copy link
Member Author

Good catch fixed.

@mtscout6
Copy link
Member Author

May not be an issue now that 2.5.2 is out, but I haven't tested yet.

@taion
Copy link
Member

taion commented Jun 14, 2015

I don't see 2.5.2 on NPM yet. Should we merge this or hold off?

@AlexKVal
Copy link
Member

ref jsx-eslint/eslint-plugin-react#110 (comment)

@taion can you merge it please if in a 30 minutes there is still no 2.5.2 on npmjs ?
I'm going to sleep, so I cannot do it. 🍒

@AlexKVal
Copy link
Member

Update: jsx-eslint/eslint-plugin-react#110 (comment)
So I presume there is no need to - of course if it fixes the warnings 😄

@AlexKVal
Copy link
Member

Yeah. 2.5.2 is out and it works fine. I'm closing this PR.

@AlexKVal AlexKVal closed this Jun 14, 2015
@mtscout6 mtscout6 deleted the lock-eslint-plugin-react branch June 15, 2015 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants