Skip to content

Commit c8c9df3

Browse files
justin808justin808
committed
fixes
1 parent 45b512c commit c8c9df3

File tree

2 files changed

+47
-102
lines changed

2 files changed

+47
-102
lines changed

‎.github/workflows/deploy-to-control-plane.yml‎

Lines changed: 46 additions & 101 deletions
Original file line numberDiff line numberDiff line change
@@ -46,14 +46,29 @@ jobs:
4646
issues: write
4747

4848
steps:
49+
# Initial checkout only for pull_request and push events
50+
- name: Checkout code
51+
if: github.event_name == 'pull_request' || github.event_name == 'push'
52+
uses: actions/checkout@v4
53+
with:
54+
fetch-depth: 0
55+
ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.ref }}
56+
57+
# Basic checkout for other events (workflow_dispatch, issue_comment)
58+
# We'll do proper checkout after getting PR info
59+
- name: Initial checkout
60+
if: github.event_name == 'workflow_dispatch' || github.event_name == 'issue_comment'
61+
uses: actions/checkout@v4
62+
with:
63+
fetch-depth: 0
64+
4965
- name: Get PR HEAD Ref
5066
id: getRef
5167
env:
5268
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
5369
run: |
54-
# Set PR number based on event type
5570
if [[ "${{ github.event_name }}" == "workflow_dispatch" ]]; then
56-
PR_NUMBER="${{ github.event.inputs.pr_number }}"
71+
PR_NUMBER="${{ github.event.inputs.pr }}"
5772
elif [[ "${{ github.event_name }}" == "issue_comment" ]]; then
5873
PR_NUMBER="${{ github.event.issue.number }}"
5974
elif [[ "${{ github.event_name }}" == "pull_request" ]]; then
@@ -68,18 +83,18 @@ jobs:
6883
exit 1
6984
fi
7085
fi
71-
86+
7287
if [[ -z "$PR_NUMBER" ]]; then
7388
echo "Error: Could not determine PR number"
7489
exit 1
7590
fi
76-
91+
7792
# Set environment variables
7893
echo "PR_NUMBER=$PR_NUMBER" >> $GITHUB_ENV
7994
echo "APP_NAME=qa-react-webpack-rails-tutorial-pr-$PR_NUMBER" >> $GITHUB_ENV
8095
8196
# Get PR data using GitHub CLI
82-
PR_DATA=$(gh pr view $PR_NUMBER --repo ${{ github.repository }} --json headRefName,headRefOid)
97+
PR_DATA=$(gh pr view $PR_NUMBER --repo shakacode/react-webpack-rails-tutorial --json headRefName,headRefOid)
8398
if [[ $? -eq 0 ]]; then
8499
echo "PR_REF=$(echo $PR_DATA | jq -r .headRefName)" >> $GITHUB_OUTPUT
85100
echo "PR_SHA=$(echo $PR_DATA | jq -r .headRefOid)" >> $GITHUB_ENV
@@ -88,42 +103,47 @@ jobs:
88103
exit 1
89104
fi
90105
91-
- uses: actions/checkout@v4
106+
- name: Checkout PR code
107+
if: github.event_name == 'workflow_dispatch' || github.event_name == 'issue_comment'
108+
uses: actions/checkout@v4
92109
with:
93110
fetch-depth: 0
94-
# 1. For comment/manual: use branch from PR number lookup
95-
# 2. For PR events: use the PR's branch
96-
ref: ${{ steps.getRef.outputs.PR_REF || (github.event_name == 'pull_request' && github.event.pull_request.head.ref) }}
97-
98-
- name: Setup Environment
99-
uses: ./.github/actions/setup-environment
100-
with:
101-
token: ${{ env.CPLN_TOKEN }}
102-
org: ${{ env.CPLN_ORG }}
111+
ref: ${{ steps.getRef.outputs.PR_SHA }}
103112

104113
- name: Check if Review App Exists
105114
id: check-app
115+
if: github.event_name == 'pull_request'
116+
env:
117+
CPLN_TOKEN: ${{ secrets.CPLN_TOKEN_STAGING }}
106118
run: |
107119
if ! cpflow exists -a ${{ env.APP_NAME }}; then
108120
echo "No review app exists for this PR"
109121
exit 0
110122
fi
111-
echo "app_exists=true" >> $GITHUB_OUTPUT
112123
113124
- name: Validate Deployment Request
114125
id: validate
115126
run: |
116-
if [[ "${{ github.event_name }}" == "pull_request" && "${{ steps.check-app.outputs.app_exists }}" == "true" ]] || \
117-
[[ "${{ github.event_name }}" == "workflow_dispatch" ]] || \
118-
[[ "${{ github.event_name }}" == "issue_comment" && "${{ github.event.comment.body }}" == "/deploy-review-app" ]] || \
119-
[[ "${{ github.event_name }}" == "push" ]]; then
120-
echo "SHOULD_DEPLOY=true" >> $GITHUB_ENV
121-
else
122-
echo "SHOULD_DEPLOY=false" >> $GITHUB_ENV
127+
if ! [[ "${{ github.event_name }}" == "workflow_dispatch" || \
128+
("${{ github.event_name }}" == "issue_comment" && "${{ github.event.comment.body }}" == "/deploy-review-app") || \
129+
"${{ github.event_name }}" == "pull_request" ]]; then
123130
echo "Skipping deployment - not a valid trigger (event: ${{ github.event_name }})"
124131
exit 0
125132
fi
126133
134+
- name: Create Initial Comment
135+
id: create-comment
136+
uses: actions/github-script@v7
137+
with:
138+
script: |
139+
const result = await github.rest.issues.createComment({
140+
owner: context.repo.owner,
141+
repo: context.repo.repo,
142+
issue_number: process.env.PR_NUMBER,
143+
body: '🚀 Starting deployment process...\n\n' + process.env.CONSOLE_LINK
144+
});
145+
core.setOutput('comment-id', result.data.id);
146+
127147
- name: Set Deployment URLs
128148
id: set-urls
129149
uses: actions/github-script@v7
@@ -146,81 +166,6 @@ jobs:
146166
'https://console.cpln.io/console/org/' + process.env.CPLN_ORG + '/gvc/' + process.env.APP_NAME + '/-info)'
147167
);
148168
149-
- name: Create Initial Comment
150-
id: create-comment
151-
uses: actions/github-script@v7
152-
with:
153-
script: |
154-
const result = await github.rest.issues.createComment({
155-
owner: context.repo.owner,
156-
repo: context.repo.repo,
157-
issue_number: process.env.PR_NUMBER,
158-
body: '🚀 Deploying Review App...\n\n' + process.env.CONSOLE_LINK
159-
});
160-
return result.data.id;
161-
162-
- name: Set Comment ID
163-
run: echo "COMMENT_ID=${{ fromJSON(steps.create-comment.outputs.result).commentId }}" >> $GITHUB_ENV
164-
165-
- name: Initialize Deployment
166-
id: init-deployment
167-
uses: actions/github-script@v7
168-
with:
169-
script: |
170-
async function getWorkflowUrl(runId) {
171-
const jobs = await github.rest.actions.listJobsForWorkflowRun({
172-
owner: context.repo.owner,
173-
repo: context.repo.repo,
174-
run_id: runId
175-
});
176-
177-
const currentJob = jobs.data.jobs.find(job => job.status === 'in_progress');
178-
const jobId = currentJob?.id;
179-
180-
if (!jobId) {
181-
console.log('Warning: Could not find current job ID');
182-
return `${process.env.GITHUB_SERVER_URL}/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId}`;
183-
}
184-
185-
return `${process.env.GITHUB_SERVER_URL}/${context.repo.owner}/${context.repo.repo}/actions/runs/${runId}/job/${jobId}`;
186-
}
187-
188-
// Create initial deployment comment
189-
const comment = await github.rest.issues.createComment({
190-
owner: context.repo.owner,
191-
repo: context.repo.repo,
192-
issue_number: process.env.PR_NUMBER,
193-
body: '⏳ Initializing deployment...'
194-
});
195-
196-
// Create GitHub deployment
197-
const deployment = await github.rest.repos.createDeployment({
198-
owner: context.repo.owner,
199-
repo: context.repo.repo,
200-
ref: context.sha,
201-
environment: 'review',
202-
auto_merge: false,
203-
required_contexts: []
204-
});
205-
206-
const workflowUrl = await getWorkflowUrl(context.runId);
207-
208-
return {
209-
deploymentId: deployment.data.id,
210-
commentId: comment.data.id,
211-
workflowUrl
212-
};
213-
214-
- name: Set comment ID and workflow URL
215-
run: |
216-
echo "COMMENT_ID=${{ fromJSON(steps.init-deployment.outputs.result).commentId }}" >> $GITHUB_ENV
217-
echo "WORKFLOW_URL=${{ fromJSON(steps.init-deployment.outputs.result).workflowUrl }}" >> $GITHUB_ENV
218-
219-
- name: Set commit hash
220-
run: |
221-
FULL_COMMIT="${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || steps.getRef.outputs.PR_SHA || github.sha }}"
222-
echo "COMMIT_HASH=${FULL_COMMIT:0:7}" >> $GITHUB_ENV
223-
224169
- name: Update Status - Building
225170
uses: actions/github-script@v7
226171
with:
@@ -237,7 +182,7 @@ jobs:
237182
await github.rest.issues.updateComment({
238183
owner: context.repo.owner,
239184
repo: context.repo.repo,
240-
comment_id: process.env.COMMENT_ID,
185+
comment_id: ${{ steps.create-comment.outputs.comment-id }},
241186
body: buildingMessage
242187
});
243188
@@ -269,7 +214,7 @@ jobs:
269214
await github.rest.issues.updateComment({
270215
owner: context.repo.owner,
271216
repo: context.repo.repo,
272-
comment_id: process.env.COMMENT_ID,
217+
comment_id: ${{ steps.create-comment.outputs.comment-id }},
273218
body: deployingMessage
274219
});
275220
@@ -329,6 +274,6 @@ jobs:
329274
await github.rest.issues.updateComment({
330275
owner: context.repo.owner,
331276
repo: context.repo.repo,
332-
comment_id: process.env.COMMENT_ID,
277+
comment_id: ${{ steps.create-comment.outputs.comment-id }},
333278
body: isSuccess ? successMessage : failureMessage
334279
});

‎.github/workflows/help-command.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
uses: actions/checkout
3232

3333
- name: Process Help Command
34-
uses: shakacode/react-webpack-rails-tutorial/.github/actions/help-command@justin808-more-work-on-review-apps-2
34+
uses: ./.github/actions/help-command
3535
with:
3636
github-token: ${{ secrets.GITHUB_TOKEN }}
3737
issue-number: ${{ github.event.inputs.issue-number }}

0 commit comments

Comments
 (0)