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

[new] add react-redux@4.0.0 with npm auto-update #5980

Merged
merged 1 commit into from Oct 22, 2015

Conversation

Amomo
Copy link
Contributor

@Amomo Amomo commented Oct 20, 2015

Hi @Piicksarn

For #5947, I add react-redux at v4.0.0 and add npm auto-update config in its package.json.
repo: https://github.com/rackt/react-redux

Would you mind checking this PR for me? Thank you~ :-D

  • Not found on cdnjs repo
  • No already exist issue and PR
  • Notable popularity : Watch 98, Star 932, Fork 98
  • Project has public repository on famous online hosting platform (or been hosted on npm)
  • Has valid tags for each versions (for git auto-update)
  • Auto-update setup
  • Auto-update target/source is valid.
  • Auto-update filemap is correct.

@gaearon
Copy link
Contributor

gaearon commented Oct 20, 2015

Thank you for this. :-)

@Piicksarn
Copy link
Contributor

@Amomo, sorry for my delay!
The pr is fine but need rebase to the master!
Thank you 😃

@Amomo
Copy link
Contributor Author

Amomo commented Oct 22, 2015

@Piicksarn I have rebased it, could you check again? Thanks~

@Piicksarn
Copy link
Contributor

It's fine!
@PeterDaveHello, would you mind to have a look at the pr?
thank you!

PeterDaveHello added a commit that referenced this pull request Oct 22, 2015
[new] add react-redux@4.0.0 with npm auto-update
@PeterDaveHello PeterDaveHello merged commit 443cb14 into cdnjs:master Oct 22, 2015
@Amomo Amomo deleted the react-redux branch November 6, 2015 18:13
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

4 participants