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: adding commit fixes and descriptions to the rules' properties #58

Merged
merged 1 commit into from
Feb 24, 2021

Conversation

solal-snyk
Copy link
Contributor

COD-114 Storing automatic fix examples on our DB

πŸ—„οΈ Example commit fixes need to be stored in our database.

  • Ready for review
  • Linked to Jira ticket

What does this PR do?

βœ… Exporting the ExampleCommitFix interface to make it accessible to the Sarif Converter.
βœ… Updating the properties object on rules instances of the Sarif Converter to be able to store them.
βœ… Updating the Sarif Converter spec to test for the presence of the exemple ExampleCommitFix.

Where should the reviewer start?

  • Check the new RuleProperty type and its use.
  • Run the tests/sarif_converter.spec.ts test.

How should this be manually tested?

  • Run the tests/sarif_converter.spec.ts test.

COD-114 Storing automatic fix examples on our DB

πŸ—„οΈ Example commit fixes need to be stored in our database.

βœ… Exporting the `ExampleCommitFix` interface to make it accessible to the Sarif Converter.
βœ… Updating the `properties` object on `rules` instances of the Sarif Converter to be able to store them.
βœ… Updating the Sarif Converter spec to test for the presence of the exemple `ExampleCommitFix`.
@solal-snyk solal-snyk requested a review from a team as a code owner February 24, 2021 13:05
@solal-snyk
Copy link
Contributor Author

Referring to the now closed:
#56

Copy link
Contributor

@orkamara orkamara left a comment

Choose a reason for hiding this comment

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

πŸŽ–οΈ

@solal-snyk solal-snyk merged commit 476d96d into master Feb 24, 2021
@solal-snyk solal-snyk deleted the feat/storing-fixes-in-db branch February 24, 2021 20:17
@snyksec
Copy link

snyksec commented Feb 24, 2021

πŸŽ‰ This PR is included in version 3.1.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