-
Notifications
You must be signed in to change notification settings - Fork 903
feat: support workflow permissions in repository #2309
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: support workflow permissions in repository #2309
Conversation
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
92f3222 to
3a2c40d
Compare
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
|
Is there something missing from the PR to be released please? |
|
@kfcampbell Sorry to ping you, but is it possible to get a review and move forward with this PR? |
|
@kfcampbell can we get a review please ? I don't know what is blocking but it's mandatory for my use case :) Regards, |
|
Would like some movement on this as well. Currently I’m running the provider compiled from the fork. |
|
@M0NsTeRRR perhaps can you open new Issue and link there this PR? |
|
fix #2584 |
|
It looks like the related issue #2584 is in the backlog but this PR is marked as in review. @nickfloyd @kfcampbell Could we get a review of this please? |
|
It would be nice to have this feature merged in. |
Signed-off-by: Ludovic Ortega <ludovic.ortega@adminafk.fr>
nickfloyd
left a comment
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.
❤️
Resolves #1266
Resolves #2584
Relates to: #1228
After the change?
github_workflow_repository_permissionsIn

settings/actionsPull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!