Skip to content

[Fix] Add Dependency Review Step to Workflow #65

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

Merged
merged 1 commit into from
Jan 24, 2024
Merged

[Fix] Add Dependency Review Step to Workflow #65

merged 1 commit into from
Jan 24, 2024

Conversation

coleleavitt
Copy link
Contributor

Title: Add Dependency Review Step to Workflow

Description:

This PR introduces a crucial update to our GitHub Actions workflow. We've added the Dependency Review step to both the dependency-review-pr and dependency-review-scheduled jobs in our .github/workflows/dependency-review.yml file.

The Dependency Review step uses actions/dependency-review-action@v3 and now includes the necessary base-ref and head-ref parameters. For pull request events, these references are dynamically set based on the pull request details or default to the 'main' branch. For scheduled events, the base-ref is set to 'main', and the head-ref is set to the current ref.

These changes ensure that our dependencies are reviewed correctly, whether the workflow is triggered by a pull request or a scheduled event. This will help us maintain the health and security of our project by keeping our dependencies up-to-date and identifying potential vulnerabilities.

Please review the changes and provide any feedback.

…pr job with the base-ref and head-ref parameters.
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@kzndotsh kzndotsh merged commit e4e4e5b into main Jan 24, 2024
@kzndotsh kzndotsh deleted the dev branch March 24, 2024 20:36
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.

2 participants