Fix gen-commands PATH resolution in GitHub Actions workflow - #11
Closed
gregmankes with Copilot wants to merge 5 commits into
Closed
Fix gen-commands PATH resolution in GitHub Actions workflow#11gregmankes with Copilot wants to merge 5 commits into
gregmankes with Copilot wants to merge 5 commits into
Conversation
Co-authored-by: gregmankes <8827475+gregmankes@users.noreply.github.com>
…ound error Co-authored-by: gregmankes <8827475+gregmankes@users.noreply.github.com>
…dback Co-authored-by: gregmankes <8827475+gregmankes@users.noreply.github.com>
Comment on lines
+30
to
+36
| runs-on: ubuntu-latest | ||
| needs: verify-generation | ||
| steps: | ||
| - name: Checkout repository | ||
| uses: actions/checkout@v5 | ||
| - name: Build the cli binary | ||
| run: make build |
There was a problem hiding this comment.
No explicit GITHUB_TOKEN permissions found at the workflow or job level. Add a permissions: block at the workflow root (applies to all jobs) or per job with least privilege (e.g., contents: read and only specific writes like pull-requests: write if needed).
🥳 Fixed in commit ebd44de 🥳
Comment on lines
+12
to
+28
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout repository | ||
| uses: actions/checkout@v5 | ||
| - name: Run commands generation command | ||
| run: make gen | ||
| - name: Verify no changes were generated | ||
| run: | | ||
| # Check if there are any uncommitted changes in the working directory | ||
| if [ -z "$(git status --porcelain)" ]; then | ||
| echo "::notice::Generated files are up to date. No changes detected." | ||
| else | ||
| echo "::error::Detected uncommitted changes after running the generation command. Please run the generator and commit the changes." | ||
| git status | ||
| git diff | ||
| exit 1 | ||
| fi |
There was a problem hiding this comment.
No explicit GITHUB_TOKEN permissions found at the workflow or job level. Add a permissions: block at the workflow root (applies to all jobs) or per job with least privilege (e.g., contents: read and only specific writes like pull-requests: write if needed).
🍰 Fixed in commit ebd44de 🍰
Co-authored-by: gregmankes <8827475+gregmankes@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix GitHub Actions workflow by adding Go bin to PATH
Fix gen-commands PATH resolution in GitHub Actions workflow
Feb 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
verify-generationjob was failing becausegen-commandsinstalled viago installwas not in PATH whenmake genexecuted.Changes
actions/setup-go@v5withgo-version-file: 'go.mod'to ensure Go 1.25.3 is available$HOME/go/binto$GITHUB_PATHbefore runningmake genso binaries installed bygo installare discoverablepermissions: contents: readto follow least-privilege security practicesThe Makefile's
installtarget clones the temporal CLI repo and runsgo install ./cmd/gen-commands, which installs to$GOPATH/bin(typically~/go/bin). This directory is not in the default GitHub Actions runner PATH.Original prompt
This pull request was created from Copilot chat.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.