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

Add monitoring version of package.json #11528

Merged
merged 1 commit into from
Feb 12, 2016

Conversation

andy-armstrong
Copy link
Contributor

This commit adds a version of package.json to be used solely for monitoring by Gemnasium. See the README.md file for details as to the purpose of this. I verified that all of the libraries in this file exist in npm, although some of them are so old that their particular versions are not available. It will be interesting to see whether Gemnasium will be able to deal with this, or whether I'll have to remove those items.

I originally intended to list all the libraries and just comment out the ones that are not available, but unfortunately JSON does not support comments and npm strictly enforces this. So as not to lose this information, I've also added a package.txt file with the raw information about every vendor library I could find, along with versions (where available).

@e0d I'm not sure how best to test this with Gemnasium. Is the simplest thing to do to just merge the commit (upon approval) and see what happens? I'm slightly nervous that it will choke on some of the very old library references.

@e0d @cahrens Please review.

FYI @dan-f @AlasdairSwan @benpatterson

@andy-armstrong andy-armstrong force-pushed the andya/add-js-monitoring-package branch 2 times, most recently from 705d098 to cc1e59c Compare February 12, 2016 06:04
@andy-armstrong
Copy link
Contributor Author

@feanil FYI

@feanil
Copy link
Contributor

feanil commented Feb 12, 2016

👍

2 similar comments
@cahrens
Copy link

cahrens commented Feb 12, 2016

👍

@dan-f
Copy link
Contributor

dan-f commented Feb 12, 2016

👍

andy-armstrong added a commit that referenced this pull request Feb 12, 2016
@andy-armstrong andy-armstrong merged commit b46648e into master Feb 12, 2016
@andy-armstrong andy-armstrong deleted the andya/add-js-monitoring-package branch February 12, 2016 18:40
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.

4 participants