-
Notifications
You must be signed in to change notification settings - Fork 10
✨ feat: add dirty-waters to CI #1083
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
✨ feat: add dirty-waters to CI #1083
Conversation
all defaults here to feel how this works.
|
@randomicecube how to make progress towards merge here? Thanks! |
d4cf934
to
9228ebb
Compare
Rebasing to trigger new CI without non-allowlisted actions. |
Hey @LogFlames @monperrus |
@LogFlames currently, it's not allowed for the action to post comments (see https://github.com/chains-project/maven-lockfile/actions/runs/13898731122/job/38885195489?pr=1083#step:4:3144); I think you might have to allow this? |
@randomicecube I think it might be because this is a PR from an public repo outside the chains-project (https://docs.github.com/en/actions/security-for-github-actions/security-guides/automatic-token-authentication#permissions-for-the-github_token).
Have you been able to create PRs from forks into other projects without this issue? Maybe @algomaster99 has higher access and can find a setting? Otherwise, do you have access to create a branch on |
@LogFlames Ah, that should be it yeah! |
You can set
But try this and then let me know :) |
Continued in #1134 . |
cc @monperrus @Stamp9
Relates to INRIA/spoon#5216, chains-project/dirty-waters#37, chains-project/dirty-waters#58
Key notes:
x_to_fail
parameter: the percentage of a single non-high severity issue present among the dependencies for the CI to break. It defaults to 5%, so what should it be here?comment_on_commit
: whether the reports are allowed to be pasted as comments in the commits, in the case of high-severity issues breaking CI. Defaults to false, what do we want here?allow_pr_comments
: whether the reports are allowed to be pasted as comments in pull requests. Defaults to true, what do we want here?For more information on the action, see the wiki.