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

chore: refactor pin remediation data structure #837

Merged
merged 1 commit into from
Oct 28, 2019

Conversation

gitphill
Copy link
Contributor

@gitphill gitphill commented Oct 28, 2019

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

What does this PR do?

Refactor PinRemediation datastructure to align with UpgradeRemediation, chaning 'issues' to 'vulns'.

@gitphill gitphill requested a review from a team as a code owner October 28, 2019 16:27
@ghost ghost requested review from miiila and orsagie October 28, 2019 16:27
@lili2311
Copy link
Contributor

Please change this to fix

@gitphill gitphill force-pushed the chore/refactor-remediation-types branch from 3e8e4b4 to 31a3879 Compare October 28, 2019 16:39
@lili2311
Copy link
Contributor

Please update the test fixture called response-with-remediation.json to reflect the new data format for pins

Copy link
Contributor

@lili2311 lili2311 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Locally it breaks for me, please re-check?

Running against local latest registry and this branch

@lili2311
Copy link
Contributor

Works ok locally

@lili2311 lili2311 force-pushed the chore/refactor-remediation-types branch 2 times, most recently from 49f5330 to 6891589 Compare October 28, 2019 17:43
package.json Outdated
@@ -122,9 +123,6 @@
"tslint": "^5.14.0",
"typescript": "^3.4.1"
},
"bundledDependencies": [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please rebase over latest master?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was removed a few days ago

@gitphill gitphill force-pushed the chore/refactor-remediation-types branch 4 times, most recently from 7292b74 to 25533ea Compare October 28, 2019 18:31
Align Pin and Upgrade data structures, using 'vuln' instead of 'issues'.
@gitphill gitphill force-pushed the chore/refactor-remediation-types branch from 25533ea to 7670c92 Compare October 28, 2019 20:25
@gitphill gitphill merged commit d00a52a into master Oct 28, 2019
@gitphill gitphill deleted the chore/refactor-remediation-types branch October 28, 2019 20:59
@snyksec
Copy link

snyksec commented Oct 28, 2019

🎉 This PR is included in version 1.239.3 🎉

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