move github contexts to env

matifali · matifali · commit 875df5b95311 · 2025-01-22T19:03:00.000Z
diff --git a/.github/workflows/contrib.yaml b/.github/workflows/contrib.yaml
@@ -55,6 +55,10 @@ jobs:
     if: github.event_name == 'pull_request' && github.event.pull_request.user.login == 'dependabot[bot]' && github.repository == 'coder/coder' && github.event.pull_request.merged
     steps:
       - name: Send Slack notification
+        env:
+          PR_URL: ${{github.event.pull_request.html_url}}
+          PR_TITLE: ${{github.event.pull_request.title}}
+          PR_NUMBER: ${{github.event.pull_request.number}}
         run: |
           curl -X POST -H 'Content-type: application/json' \
           --data '{
@@ -63,7 +67,7 @@ jobs:
                 "type": "header",
                 "text": {
                   "type": "plain_text",
-                  "text": ":pr-merged: Auto merged Dependabot PR #${{ github.event.pull_request.number }}",
+                  "text": ":pr-merged: Auto merged Dependabot PR #${{ env.PR_NUMBER }}",
                   "emoji": true
                 }
               },
@@ -72,7 +76,7 @@ jobs:
                 "fields": [
                   {
                     "type": "mrkdwn",
-                    "text": "${{ github.event.pull_request.title }}"
+                    "text": "${{ env.PR_TITLE }}"
                   }
                 ]
               },
@@ -85,7 +89,7 @@ jobs:
                       "type": "plain_text",
                       "text": "View PR"
                     },
-                    "url": "${{ github.event.pull_request.html_url }}"
+                    "url": "${{ env.PR_URL }}"
                   }
                 ]
               }

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,10 @@ jobs:`
`55`	`55`	`if: github.event_name == 'pull_request' && github.event.pull_request.user.login == 'dependabot[bot]' && github.repository == 'coder/coder' && github.event.pull_request.merged`
`56`	`56`	`steps:`
`57`	`57`	`- name: Send Slack notification`
	`58`	`+ env:`
	`59`	`+ PR_URL: ${{github.event.pull_request.html_url}}`
	`60`	`+ PR_TITLE: ${{github.event.pull_request.title}}`
	`61`	`+ PR_NUMBER: ${{github.event.pull_request.number}}`
`58`	`62`	`run: \|`
`59`	`63`	`curl -X POST -H 'Content-type: application/json' \`
`60`	`64`	`--data '{`
`@@ -63,7 +67,7 @@ jobs:`
`63`	`67`	`"type": "header",`
`64`	`68`	`"text": {`
`65`	`69`	`"type": "plain_text",`
`66`		`- "text": ":pr-merged: Auto merged Dependabot PR #${{ github.event.pull_request.number }}",`
	`70`	`+ "text": ":pr-merged: Auto merged Dependabot PR #${{ env.PR_NUMBER }}",`
`67`	`71`	`"emoji": true`
`68`	`72`	`}`
`69`	`73`	`},`
`@@ -72,7 +76,7 @@ jobs:`
`72`	`76`	`"fields": [`
`73`	`77`	`{`
`74`	`78`	`"type": "mrkdwn",`
`75`		`- "text": "${{ github.event.pull_request.title }}"`
	`79`	`+ "text": "${{ env.PR_TITLE }}"`
`76`	`80`	`}`
`77`	`81`	`]`
`78`	`82`	`},`
`@@ -85,7 +89,7 @@ jobs:`
`85`	`89`	`"type": "plain_text",`
`86`	`90`	`"text": "View PR"`
`87`	`91`	`},`
`88`		`- "url": "${{ github.event.pull_request.html_url }}"`
	`92`	`+ "url": "${{ env.PR_URL }}"`
`89`	`93`	`}`
`90`	`94`	`]`
`91`	`95`	`}`