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

chore: Add dependency-review workflow for pull requests #7

Merged
merged 1 commit into from
Mar 29, 2024

Conversation

joonas
Copy link
Member

@joonas joonas commented Mar 29, 2024

Feature or Problem

GitHub recently announced the addition of OpenSSF Scorecard support in the dependency-review GitHub Action, which brought this action to my attention.

Having looked at it, this looks like it could be a useful addition for the PR pipeline in order to help maintain awareness of the state of dependencies that are being introduced/changed.

Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

Signed-off-by: Joonas Bergius <joonas@cosmonic.com>
@joonas joonas requested a review from protochron March 29, 2024 17:45
# https://github.com/actions/dependency-review-action?tab=readme-ov-file#configuration-options
fail-on-severity: critical

comment-summary-in-pr: always
Copy link
Member Author

Choose a reason for hiding this comment

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

We could switch this to on-failure, but I figured always would be nice to start with to see what this looks like in practice.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, let's see what it looks like for now.

- name: Dependency Review
uses: actions/dependency-review-action@v4
with:
config-file: './.github/dependency-review-config.yml'
Copy link
Member Author

Choose a reason for hiding this comment

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

We might want to consider pointing this to an external reference at some point via something like this:

# Syntax for an external file: OWNER/REPOSITORY/FILENAME@BRANCH
config-file: 'wasmcloud/wasmcloud-operator/.github/dependency-review-config.yml@main'

But for now I think it should be fine as it is.

@joonas joonas merged commit 88369e3 into wasmCloud:main Mar 29, 2024
5 checks passed
@joonas joonas deleted the chore/add-dependency-review branch March 29, 2024 21:06
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