Skip to content

Commit 38eacb5

Browse files
authored
Merge pull request #31 from Code-dot-mil/hshah-mitre-patch-1
Upgrade checkout action from v3 to v4
2 parents 728b41e + 53b54ae commit 38eacb5

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

.github/workflows/deploy.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@ jobs:
2020
if: github.event_name == 'pull_request'
2121
steps:
2222
- name: Checkout code
23-
uses: actions/checkout@v3
23+
uses: actions/checkout@v4
2424
- name: configure aws credentials
25-
uses: aws-actions/configure-aws-credentials@v1.7.0
25+
uses: aws-actions/configure-aws-credentials@v4
2626
with:
2727
role-to-assume: arn:aws:iam::677043464939:role/GitHubAction-AssumeRoleWithAction
2828
role-session-name: GitHub_to_AWS_via_FederatedOIDC
@@ -44,8 +44,6 @@ jobs:
4444
ls -al .github
4545
cat .github/auto_assign.yml || echo "No auto_assign.yml found"
4646
47-
- uses: actions/checkout@v3
48-
4947
- name: Assign assignee via GitHub API
5048
env:
5149
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -82,7 +80,7 @@ jobs:
8280
if: github.ref != 'refs/heads/main' # Run for non-main branches
8381
steps:
8482
- name: Checkout code
85-
uses: actions/checkout@v3
83+
uses: actions/checkout@v4
8684

8785
- name: Set up Python
8886
uses: actions/setup-python@v4
@@ -118,7 +116,7 @@ jobs:
118116
if: github.event_name == 'pull_request' && github.ref != 'refs/heads/main' # Run for non-main branches
119117
steps:
120118
- name: Checkout code
121-
uses: actions/checkout@v3
119+
uses: actions/checkout@v4
122120

123121
#- name: Get PR number from GitHub API
124122
# id: pr
@@ -200,3 +198,6 @@ jobs:
200198
$PR_API_URL | jq '.[0].number')
201199
echo "after pr_number=$PR_NUMBER"
202200
aws s3 sync s3://$S3_BUCKET_NAME/mergeRequest/$PR_NUMBER s3://$S3_BUCKET_NAME/production/
201+
echo "deploy complete..."
202+
# Delete contents of the pr folder
203+
aws s3 rm s3://$S3_BUCKET_NAME/mergeRequest/$PR_NUMBER --recursive

0 commit comments

Comments
 (0)