-
Notifications
You must be signed in to change notification settings - Fork 779
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 dependabot.yml
configuration
#4179
Conversation
Created #4180 to update or replace all the pins |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Left one suggestions
# Workflows are not allowed to edit workflows. As result, we need to prevent Prettier from formatting them. | ||
- name: Prevent workflows from being formatted | ||
run: echo ".github" >> .prettierignore | ||
- run: npm run fmt | ||
# Prevent the prettierignore change from being committed. | ||
- run: git checkout .prettierignore | ||
- uses: stefanzweifel/git-auto-commit-action@v4 | ||
- uses: stefanzweifel/git-auto-commit-action@8756aa072ef5b4a080af5dc8fef36c5d586e521d # tag=v5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume you did a security review on these. Is it worth sticking a comment in here to document that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checking in a tag + pinned SHA like this is exactly comparable to checking in a version range + lockfile for an npm dependency - why require comments here but not there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for leaving the comment breadcrumb for #3771
This PR adds dependabot configuration for automatically updating dependencies for NPM and GitHub Actions. While I was updating the actions for sha pinning, I also updated to not use
actions/cache
for npm cache and instead useactions/setup-node
as is recommended, and updated the actions to run on node 20.