Skip to content

ci: add dependency-review job#3884

Merged
mcollina merged 2 commits intofastify:mainfrom
Fdawgs:chore/dependency-review
May 8, 2022
Merged

ci: add dependency-review job#3884
mcollina merged 2 commits intofastify:mainfrom
Fdawgs:chore/dependency-review

Conversation

@Fdawgs
Copy link
Member

@Fdawgs Fdawgs commented May 6, 2022

closes #3883

After merging this, someone with permissions will need to remove Snyk hook from https://github.com/fastify/fastify/settings/hooks

Checklist

jobs:
dependency-review:
name: Dependency Review
if: github.event_name == 'pull_request'
Copy link
Member Author

@Fdawgs Fdawgs May 6, 2022

Choose a reason for hiding this comment

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

Only runs on pull_request events at present. See actions/dependency-review-action#30

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@xtx1130 xtx1130 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link

github-actions bot commented May 9, 2023

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace Snyk webhook with GitHub's own dependency review action

5 participants