Auto codespaces feedback round 1 (#54690)
This commit is contained in:
7
.github/workflows/codespace-review-check.yml
vendored
7
.github/workflows/codespace-review-check.yml
vendored
@@ -58,8 +58,7 @@ jobs:
|
||||
fi
|
||||
|
||||
# https://stackoverflow.com/a/70716837
|
||||
# This might not need `| sed 's/"/\\"/g'`
|
||||
matrix=$(echo "$branches" | jq -scR 'split("\n") | map(select(. != ""))' | sed 's/"/\\"/g')
|
||||
matrix=$(echo "$branches" | jq -scR 'split("\n") | map(select(. != ""))')
|
||||
echo "- Matrix: $matrix"
|
||||
echo "matrix=$matrix" >> $GITHUB_OUTPUT
|
||||
|
||||
@@ -76,7 +75,7 @@ jobs:
|
||||
matrix:
|
||||
value: ${{ fromJSON(needs.codespace-review-check-find.outputs.matrix) }}
|
||||
runs-on: ubuntu-latest
|
||||
if: ${{ github.repository == 'github/docs-internal' }}
|
||||
if: ${{ github.repository == 'github/docs-internal' && needs.codespace-review-check-find.outputs.matrix }}
|
||||
env:
|
||||
repo: github/docs-internal
|
||||
steps:
|
||||
@@ -84,6 +83,8 @@ jobs:
|
||||
|
||||
- name: Find the pull request
|
||||
id: findPr
|
||||
env:
|
||||
GH_TOKEN: ${{ secrets.DOCS_BOT_PAT_CODESPACE }}
|
||||
run: |
|
||||
echo "Looking up pull request"
|
||||
echo "- Branch: ${{ matrix.value }}"
|
||||
|
||||
4
.github/workflows/codespace-review-up.yml
vendored
4
.github/workflows/codespace-review-up.yml
vendored
@@ -214,10 +214,6 @@ jobs:
|
||||
After 4 hours, you can reactivate your codespace by applying the https://github.com/${{ env.REPO }}/labels/extend-codespace label to the pull request.
|
||||
If the label is already applied, you can remove and reapply the label to reactivate your codespace.
|
||||
|
||||
#### Maximum of 100 _active_ codespaces per user
|
||||
|
||||
Your codespace may be deleted if the limit is reached.
|
||||
|
||||
<details><summary>Table of review links</summary>
|
||||
|
||||
${{ steps.changes.outputs.changesTable && 'The table shows the files in the `content` directory that were changed in this pull request. This helps you review your changes on the review server. Changes to the `data` directory are not included in this table.' || '' }}
|
||||
|
||||
Reference in New Issue
Block a user