Skip to content

Commit

Permalink
Dat 17763 (#132)
Browse files Browse the repository at this point in the history
* πŸ”§ (test.yml): update workflow to trigger on pull_request_target event instead of pull_request for better control and permissions management
πŸ”§ (test.yml): add permissions configuration for contents and pull-requests to ensure proper access rights
πŸ”§ (test.yml): introduce an authorization step to differentiate between internal and external pull requests for enhanced security
πŸ”§ (test.yml): adjust workflow dependencies to ensure authorization step runs before build-test job for accurate execution

* βœ… (test.yml): add dependency between jobs to ensure authorization job runs before integration test job
  • Loading branch information
jandroav authored Jun 14, 2024
1 parent 3640c66 commit 77094f8
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
permissions:
contents: write
pull-requests: write

jobs:

authorize:
Expand All @@ -26,6 +26,7 @@ jobs:
secrets: inherit

integration-test:
needs: authorize
name: Test Harness for Maxdb ${{ matrix.maxdb }}
runs-on: ubuntu-latest

Expand Down

0 comments on commit 77094f8

Please sign in to comment.