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

Feat/node 6 deprecation warning #853

Merged
merged 3 commits into from
Nov 5, 2019

Conversation

dkontorovskyy
Copy link
Contributor

@dkontorovskyy dkontorovskyy commented Nov 4, 2019

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

Adds an EOL warning at the end of the CLI output for users that are still using Node v6.
From external contribution #803
https://github.com/snyk/snyk/issues/787

@dkontorovskyy dkontorovskyy requested a review from a team as a code owner November 4, 2019 12:04
@ghost ghost requested review from gitphill and lili2311 November 4, 2019 12:05
@dkontorovskyy dkontorovskyy force-pushed the feat/node-6-deprecation-warning branch 2 times, most recently from d45f240 to 4ba347e Compare November 4, 2019 12:10
@dkontorovskyy dkontorovskyy force-pushed the feat/node-6-deprecation-warning branch 2 times, most recently from 2a0911e to 5d7b30b Compare November 4, 2019 13:56
@lili2311
Copy link
Contributor

lili2311 commented Nov 4, 2019

@dkontorovskyy looks good locally, please run the copy by Rachel?

@dkontorovskyy
Copy link
Contributor Author

@lili2311 just to verify text with her? Or show how it actually looks ?

@lili2311
Copy link
Contributor

lili2311 commented Nov 5, 2019

@dkontorovskyy just to check the text yep!

@dkontorovskyy
Copy link
Contributor Author

@lili2311 done! Do you want to approve ?

@dkontorovskyy dkontorovskyy merged commit 779f0ef into master Nov 5, 2019
@dkontorovskyy dkontorovskyy deleted the feat/node-6-deprecation-warning branch November 5, 2019 13:46
@snyksec
Copy link

snyksec commented Nov 5, 2019

🎉 This PR is included in version 1.243.0 🎉

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
4 participants