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

Add OSV Scanner #6506

Conversation

btme0011
Copy link
Contributor

@btme0011 btme0011 commented Oct 5, 2023

What does this PR do?

Added OSV-Scanner while pushing and raising pull_request

Test Plan

  1. Success test -
    Screenshot 2023-10-05 at 4 10 20 PM

  2. Failure test -
    Screenshot 2023-10-05 at 3 51 20 PM

Related PRs and Issues

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@stnguyen90 stnguyen90 self-requested a review October 5, 2023 15:13
Copy link
Contributor

@stnguyen90 stnguyen90 left a comment

Choose a reason for hiding this comment

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

Great PR! 🤯 We left some comments during the review, please check them out.

.github/workflows/osv-scanner.yml Show resolved Hide resolved
@@ -0,0 +1,27 @@
name: OSV Scanner
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's change this to check-dependencies.yml and the name to Check dependencies

@stnguyen90 stnguyen90 added the hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later label Oct 27, 2023
@stnguyen90 stnguyen90 self-requested a review February 18, 2024 23:53
@gewenyu99
Copy link

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

@stnguyen90 stnguyen90 changed the base branch from main to feat-vulnerability-scanner April 24, 2024 22:57
@stnguyen90 stnguyen90 merged commit 47fc1a2 into appwrite:feat-vulnerability-scanner Apr 24, 2024
4 checks passed
This was referenced May 15, 2024
@stnguyen90 stnguyen90 changed the title feature-5232-Security-Scans-OSV-Scanner Add OSV Scanner May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants