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.

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

🔧 (test.yml): update workflow to trigger on pull_request_target event… #333

Merged
merged 2 commits into from
Jun 14, 2024

Conversation

jandroav
Copy link
Contributor

…s instead of pull_request events for better control and security

🔧 (test.yml): add permissions for contents and pull-requests to ensure proper access rights
🔧 (test.yml): add an authorization step to differentiate between internal and external pull requests for enhanced security
🔧 (test.yml): set up a dependency between the authorization step and the build-test job for proper execution order

jandroav added 2 commits June 14, 2024 09:24
…s instead of pull_request events for better control and security

🔧 (test.yml): add permissions for contents and pull-requests to ensure proper access rights
🔧 (test.yml): add an authorization step to differentiate between internal and external pull requests for enhanced security
🔧 (test.yml): set up a dependency between the authorization step and the build-test job for proper execution order
…get' event to trigger workflow on pull requests from forks for better security

🔧 (test-harness.yml): add permissions for contents and pull-requests to write to enable necessary actions in the workflow
🔧 (test-harness.yml): introduce an 'authorize' job to handle external pull requests and set up necessary permissions
🔧 (test-harness.yml): add a 'needs' directive to ensure 'authorize' job runs before 'setup' job for proper workflow execution
@jandroav jandroav merged commit 681518c into main Jun 14, 2024
2 of 3 checks passed
@jandroav jandroav deleted the DAT-17818 branch June 14, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant