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

Create semgrep for sast #1546

Closed
wants to merge 1 commit into from
Closed

Create semgrep for sast #1546

wants to merge 1 commit into from

Conversation

rparchuri
Copy link

testing static security analysis tool in github for future testing

testing static security analysis tool in github for future testing
@coveralls
Copy link

Coverage Status

Coverage remained the same at 59.311% when pulling 45c7bb6 on rparchuri-patch-2 into 7f600ea on master.

@rparchuri
Copy link
Author

@oshi97 can you help with the review here? Let me know if you have any questions.

@oshi97 oshi97 requested review from tmeyer2115, nmanu1 and cea2aj and removed request for cea2aj and nmanu1 September 2, 2021 13:33
@tmeyer2115
Copy link
Collaborator

@oshi97 can you help with the review here? Let me know if you have any questions.

Can you please provide some context on this change? Is this a Vaccine driven initiative? What does Semgrep do exactly? We don't typically merge things directly into master. We use the GitFlow process.

@rparchuri
Copy link
Author

rparchuri commented Sep 2, 2021 via email

@rparchuri rparchuri closed this Sep 10, 2021
@rparchuri rparchuri deleted the rparchuri-patch-2 branch September 10, 2021 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants