Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix dryrun action permissions #604

Merged
merged 20 commits into from
Oct 28, 2022
Merged

Fix dryrun action permissions #604

merged 20 commits into from
Oct 28, 2022

Conversation

lugi0
Copy link
Contributor

@lugi0 lugi0 commented Oct 26, 2022

Signed-off-by: Luca Giorgi lgiorgi@redhat.com

lugi0 added 3 commits October 26, 2022 17:01
@lugi0
add pull_request_target to github action
1bfc756 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
Add write permission
cf6ff5b 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
Add comment
7cf4900 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0 lugi0 changed the title add pull_request_target to dryrun github action Fix dryrun action permissions Oct 26, 2022
lugi0 added 17 commits October 26, 2022 17:12
@lugi0
give write-all permission to action
3a4158d
@lugi0
give rw permissions to action
3dcfe5a
@lugi0
add pull request target again
1d5c355
@lugi0
trigger action
e11cbd1
@lugi0
test commenting out triggers
533a90f 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
fix permission string
a83e862 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
Update dry_run.yml
20ba1e1
@lugi0
Update dry_run.yml
6de3ca4
@lugi0
Test two-step worflow to leave comment
1287e1a 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
update comment trigger
fbccea1 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
additional workflow type
5294d92 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
add another status
b8eeb9c 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
test pr_target setting
ed328f2 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
remove comments
0a0d7df 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
update workflows
e54f2fc 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
remove _target
92ba7aa 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
revert workflow name
77598bf 
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@lugi0 lugi0 merged commit 7c22193 into red-hat-data-services:master Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mwaykole mwaykole mwaykole approved these changes

@tarukumar tarukumar tarukumar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lugi0 @mwaykole @tarukumar