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

update package.json of prettify #5508

Merged
merged 1 commit into from Sep 15, 2015
Merged

Conversation

Amomo
Copy link
Contributor

@Amomo Amomo commented Aug 26, 2015

Hi @LeaYeh , From #5420, I have updated info in package.json of prettify.
From #5500, I have added license info.
old repo : https://code.google.com/p/google-code-prettify
repo : https://github.com/google/code-prettify
The repo on GitHub don't have tags, so I don't add auto-update config.
Would you mind checking this PR for me? Thank you~ :-)

  • Not found on cdnjs repo
  • No already exist issue and PR
  • Notable popularity : Watch 31, Star 443, Fork 75
  • 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.

@Amomo Amomo changed the title update repository, homepage, license, keywords info of prettify update package.json of prettify Aug 26, 2015
@Amomo
Copy link
Contributor Author

Amomo commented Aug 31, 2015

@LeaYeh I have rebased this PR to match the latest master branch.
Could you take a look? Thank you!

@Amomo Amomo force-pushed the prettify branch 3 times, most recently from fded78d to 98be679 Compare September 9, 2015 03:52
@Amomo
Copy link
Contributor Author

Amomo commented Sep 9, 2015

@LeaYeh I have rebased this PR to match the latest master branch.
Could you take a look? Thank you!

@LeaYeh
Copy link
Contributor

LeaYeh commented Sep 13, 2015

i think this pr is ok, sorry for late, and please rebase it again.

@Amomo
Copy link
Contributor Author

Amomo commented Sep 13, 2015

@LeaYeh I have rebased it, could you check again? Thank you~

@LeaYeh
Copy link
Contributor

LeaYeh commented Sep 14, 2015

@PeterDaveHello could check this pr?

@LeaYeh LeaYeh assigned PeterDaveHello and unassigned LeaYeh Sep 14, 2015
PeterDaveHello added a commit that referenced this pull request Sep 15, 2015
update package.json of prettify
@PeterDaveHello PeterDaveHello merged commit d4d6393 into cdnjs:master Sep 15, 2015
@Amomo Amomo deleted the prettify branch September 15, 2015 04:47
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