Conversation
sayakpaul
left a comment
There was a problem hiding this comment.
Left some comments. LMK if they are clear.
| "Bash(git commit*)", | ||
| "Bash(git push*)", |
There was a problem hiding this comment.
It should be able to commit specific rules to the rule file, actually:
1. NEVER modify, create, or delete files — unless the human comment contains verbatim:
COMMIT THIS (uppercase). If committing, only touch src/diffusers/ and .ai/.
How do we tackle that with this denial?
There was a problem hiding this comment.
I'm ok with keeping the claude CI read-only for now so we can get this PR merged quickly and get it into action:)
we can figure out how to allow it commit in safe way in a follow-up PR? @sayakpaul
yiyixuxu
left a comment
There was a problem hiding this comment.
thanks for working on this @paulinebm!
i left one comment
| "Bash(git commit*)", | ||
| "Bash(git push*)", |
There was a problem hiding this comment.
I'm ok with keeping the claude CI read-only for now so we can get this PR merged quickly and get it into action:)
we can figure out how to allow it commit in safe way in a follow-up PR? @sayakpaul
Co-authored-by: YiYi Xu <yixu310@gmail.com>
Co-authored-by: Pauline Bailly-Masson <155966238+paulinebm@users.noreply.github.com>
cc @sayakpaul