Skip to content

chore(security): fixed 5 vulnerabilities #599

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

Merged
merged 2 commits into from
Jul 17, 2019
Merged

Conversation

Berkmann18
Copy link
Contributor

I fixed 5 (out of 13) vulnerabilities, all of which were caused by the semantic-release's version used prior to this PR.
The 8 others are caused by codecov which are all only affecting the development so I left that aside (unless someone wants me to go ahead and try to fix those too).

Also, after running a dependency check (with npm-check -u), I've noticed that there were the following non-breaking updates which I think would be a good idea to do:

  • Patch updates: fs-extra, inquirer, nodemon [dev]
  • Minor updates (all dev dependencies): @babel/cli, @babel/core, @babel/preset-env, chai, eslint, nyc and semver.

@jimthedev
Copy link
Member

Thanks for doing this. It looks good except that on Windows the builds are breaking which is the same we were seeing when attempting this upgrade.

@Berkmann18
Copy link
Contributor Author

Berkmann18 commented Dec 30, 2018

@jimthedev Have you found out why it was breaking?
And which version bump broke it?

@Berkmann18 Berkmann18 changed the title chore(security) Fixed 5 vulnerabilities chore(security): Fixed 5 vulnerabilities Jan 14, 2019
@Berkmann18 Berkmann18 changed the title chore(security): Fixed 5 vulnerabilities chore(security): fixed 5 vulnerabilities Jan 14, 2019
@jimthedev
Copy link
Member

Looks like it was the windows builds. Something like Cmd.exe existed with code 1 but then I didn't catch the actual error and the build is gone now. If you rebase it should happen again.

Fixed 5 vulnerabilities found in `semantic-release`.
Rebased from the master branch an fixed additional vulnerabilities
@jimthedev jimthedev merged commit b24eade into commitizen:master Jul 17, 2019
@commitizen-bot
Copy link

🎉 This PR is included in version 3.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants