newrelic-quickstarts
newrelic-quickstarts copied to clipboard
Adding Snyk quickstart
Signed-off-by: Harry Kimpel [email protected]
Summary
Adding Snyk quickstart
Pre merge checklist
- [x] Did you check you NRQL syntax? - Does it work?
- [x] Did you check your dashboard image quality? - Do they look good?
- [x] Did you check that your alerts actually work?
- [x] Did you include an InstallPlan and Documentation reference?
- [x] Did you check your descriptive content for voice, tone, spelling and grammar errors?
- [x] Did you attach images of your dashboards to the PR so we can see them working?
Screenshots
Attach images of any visual changes, such as a dashboard here.
Thank you for your contribution, our team will be reviewing this shortly, please be available for any follow up questions or code review feedback!
Old PRs will be closed after 30 days of inactivity. This PR has been quiet for 14 days and is being marked as stale. Reply here to keep this PR open.
Please re-open this PR
Old PRs will be closed after 30 days of inactivity. This PR has been quiet for 14 days and is being marked as stale. Reply here to keep this PR open.
pls. re-open
Hey @harrykimpel! This is looking almost good to go, there are just a few minor things to clean up before merge.
- Could you remove the
node_modules/.yarn-integrity
andyarn.lock
files from this PR?- For this quickstart to be "installable" you'll need to create an install plan, I would recommend going with the
link
directive and setting the url to the same page as your documentation link. Here's a good example of an install plan with alink
directive.
Hi @aswanson-nr, I've deleted the yarn files and created an install plan file as suggested. For now, the link to the docs points to my personal blog. This will hopefully be migrated to an official Snyk docs page soon.
@aswanson-nr is there anything else for me to do here?
happy Wednesday @harrykimpel! My apologies for the latency here. I've only just joined the team as Product Manager as of 2 weeks ago, and so I've been slowly catching up to everything. Please allow me to the end of the week to review and then I'll have a real response for you on this. Thank you for your continued patience
happy Wednesday @harrykimpel! My apologies for the latency here. I've only just joined the team as Product Manager as of 2 weeks ago, and so I've been slowly catching up to everything. Please allow me to the end of the week to review and then I'll have a real response for you on this. Thank you for your continued patience
@seemantk no worries at all and congrats to you joining the team as PM. This is an exciting area to be in within New Relic. Just let me know if there is anything I can help with or any other feedback for such contributions.
happy Wednesday @harrykimpel! My apologies for the latency here. I've only just joined the team as Product Manager as of 2 weeks ago, and so I've been slowly catching up to everything. Please allow me to the end of the week to review and then I'll have a real response for you on this. Thank you for your continued patience
@seemantk no worries at all and congrats to you joining the team as PM. This is an exciting area to be in within New Relic. Just let me know if there is anything I can help with or any other feedback for such contributions.
@harrykimpel thank you so much! It's very exciting to be part of New Relic! I've been chatting about your PR internally and I I just need to loop in someone so that we can move forward next week. Thank you for your continued patience and bearing with me.
@harrykimpel thank you for your patience. we have some exciting product work underway that will make this integration easier, I'll love to reach out directly if you're open to that?
@harrykimpel thank you for your patience. we have some exciting product work underway that will make this integration easier, I'll love to reach out directly if you're open to that?
@seemantk sounds interesting. happy to jump on a call to discuss directly.
@harrykimpel thank you for your patience. we have some exciting product work underway that will make this integration easier, I'll love to reach out directly if you're open to that?
@seemantk sounds interesting. happy to jump on a call to discuss directly.
Excellent thank you @harrykimpel! What's the best way to connect with you? I can be e-reached via skulleen at newrelic
Old PRs will be closed after 30 days of inactivity. This PR has been quiet for 14 days and is being marked as stale. Reply here to keep this PR open.
The PR checks have run and found the following warnings:
Warning | Filepath | Line # |
---|---|---|
"permissions" field should not be used | dashboards/snyk/snyk.json | 4 |
Reference the Contributing Docs for Dashboards for more information.
The PR checks have run and found the following warnings:
Warning | Filepath | Line # |
---|---|---|
"permissions" field should not be used | dashboards/snyk/snyk.json | 4 |
Reference the Contributing Docs for Dashboards for more information.
The PR checks have run and found the following warnings:
Warning | Filepath | Line # |
---|---|---|
"permissions" field should not be used | dashboards/snyk/snyk.json | 4 |
Reference the Contributing Docs for Dashboards for more information.