-
Notifications
You must be signed in to change notification settings - Fork 710
validate dependabot configuration #10271
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
Conversation
93ff8af
to
dbf2387
Compare
Does anyone know what the whitespace check is upset about? I don't see any lines with extra spaces or tabs. (I think.) |
File not ending with |
dbf2387
to
2d397c5
Compare
Looks like it. |
cbfb33a
to
e1b6bff
Compare
Okay, I've finally got this doing what it should be. The original one wanted extra permissions I couldn't figure out how to grant, but I don't think we want or need what it wanted to do (add/update a comment, on top of the job success/failure indication). I've also verified that it succeeds when it should and fails when it should. Unfortunately, it's not answering the question that led me to make this PR: why didn't dependabot notify us of the updated actions when it's configured to do so? |
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.
cheers
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.
❤️
Indeed. I hoped something from the relevant settings page may be the reason, but it doesn't look like it https://github.com/haskell/cabal/settings/security_analysis Quick google search also doesn't bring any insights. |
Borrowed from Ubuntu (ubuntu/authd@3f9df8f) Sadly, this won't check our existing config unless I make a dummy update.
e1b6bff
to
770ceb3
Compare
Borrowed from Ubuntu
(ubuntu/authd@3f9df8f)
Sadly, this won't check our existing config unless I make a dummy update.
Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR:
master
)