Conversation
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions release workflow used to publish devcontainer features and trigger a downstream website update when a tag release is pushed.
Changes:
- Modifies the inputs passed to
actions/create-github-app-token@v3when generating the Trigganator token. - Adjusts JSON quoting/escaping for the
repository-dispatchclient-payload.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| uses: actions/create-github-app-token@v3 | ||
| with: | ||
| app-id: ${{ vars.TRIGGANATOR_ID }} | ||
| client-id: ${{ vars.TRIGGANATOR_ID }} |
There was a problem hiding this comment.
actions/create-github-app-token@v3 expects the GitHub App identifier under the app-id input (paired with private-key). Using client-id here will be ignored by the action and will cause token generation (and the subsequent repository-dispatch) to fail. Switch this input back to app-id (or whichever exact input name the action documents) and keep the variable value the same.
| client-id: ${{ vars.TRIGGANATOR_ID }} | |
| app-id: ${{ vars.TRIGGANATOR_ID }} |
| repository: helpers4/website | ||
| event-type: devcontainer-release | ||
| client-payload: '{"version": "${{ github.ref_name }}"}' | ||
| client-payload: "{\"version\": \"${{ github.ref_name }}\"}" |
There was a problem hiding this comment.
The PR description is still the default template (no summary, type of change, testing details, or linked issue). Please update it so reviewers/release tooling have the necessary context for what "Next release" changes and how it was verified.
❌ PR Validation Failed
📋 Pipeline Status
🤖 Generated by @helpers4 CI • 2026-04-25 |
Description
Please include a summary of what this PR does and why it's needed.
Type of Change
Related Issues
Closes #(issue number)
How Has This Been Tested?
Describe the tests you ran and how to reproduce them:
Checklist
Screenshots (if applicable)
Add screenshots for UI changes.
Additional Context
Add any other context about the PR here.