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

CB-14145 update dependencies #10

Merged
merged 3 commits into from Jul 10, 2018
Merged

Conversation

@brodybits
Copy link

@brodybits brodybits commented Jul 9, 2018

  • explicit nodeunit@^0.11.3 update to resolve npm audit issue on devDependencies
  • move pegjs to devDependencies, add npm run pegjs script, and update pegjs usage in README.md
  • remove extra blank lines from package.json

Requesting a quick review, would like to merge and publish barring any major issues (or worse).

Christopher J. Brody added 3 commits Jul 9, 2018
package.json
- move pegjs to devDependencies in package.json
- add pegjs script to package.json
- update README.md with easier pegjs instructions & reminder notice
@brodybits brodybits requested review from shazron and purplecabbage Jul 9, 2018
@brodybits brodybits merged commit e7646f0 into apache:master Jul 10, 2018
@brodybits brodybits deleted the brodybits:cb-14145-patch branch Jul 10, 2018
@brodybits brodybits removed request for shazron and purplecabbage Jul 30, 2018
@brodybits brodybits mentioned this pull request Jul 30, 2018
1 of 12 tasks complete
erisu added a commit that referenced this pull request Jan 15, 2019
erisu added a commit that referenced this pull request Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant