Skip to content

Commit c45c174

Browse files
authored
chore: try different auth method for request-cli-integ-test (#26525)
Try yet another authentication method for the push to the testing pipeline branch. This has worked on my fork with a PR from another fork. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 3baffd8 commit c45c174

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

.github/workflows/request-cli-integ-test.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,9 @@ jobs:
1919
persist-credentials: false
2020
- name: Find changed cli files
2121
id: changed-cli-files
22-
uses: tj-actions/changed-files@v37
22+
uses: tj-actions/changed-files@920e7b9ae1d45913fc81f86c956fee89c77d2e5e
2323
with:
24+
base_sha: ${{ github.event.pull_request.base.sha }}
2425
files_yaml: |
2526
cli:
2627
- packages/aws-cdk/bin/**
@@ -35,22 +36,22 @@ jobs:
3536
- name: Checkout
3637
uses: actions/checkout@v3
3738
with:
39+
# Needs to run with PROJEN_GITHUB_TOKEN because we need permissions to force push the branch
3840
token: ${{ secrets.PROJEN_GITHUB_TOKEN }}
3941
ref: ${{ github.event.pull_request.head.ref }}
4042
repository: ${{ github.event.pull_request.head.repo.full_name }}
4143
- name: Submit to test-pipeline
42-
# Needs to run with PROJEN_GITHUB_TOKEN because we need permissions to force push the branch
4344
run: |
4445
git config --global user.name 'aws-cdk-automation'
4546
git config --global user.email '[email protected]'
46-
git push --force --atomic ${{ github.repositoryUrl }} FETCH_HEAD:test-main-pipeline
47+
git push --force --atomic https://github.com/${{ github.repository }}.git FETCH_HEAD:test-main-pipeline
4748
- name: Explain next steps
48-
uses: thollander/actions-comment-pull-request@v2
49+
uses: thollander/actions-comment-pull-request@dadb7667129e23f12ca3925c90dc5cd7121ab57e
4950
with:
5051
message: |
5152
:arrow_right: **PR build request submitted to `test-main-pipeline`** :arrow_left:
5253
53-
A maintainer must now manually check the pipeline and add the `pr-linter/cli-integ-tested` label once the pipeline succeeds.
54+
A maintainer must now check the pipeline and add the `pr-linter/cli-integ-tested` label once the pipeline succeeds.
5455
comment_tag: request-cli-integ-test
5556
mode: recreate
5657
# Post as our automation user

0 commit comments

Comments
 (0)