Docs automation fixes#9090
Conversation
|
Newest code from mattermost has been published to preview environment for Git SHA b0850c6 |
There was a problem hiding this comment.
Pull request overview
This PR updates GitHub Actions workflows in the Mattermost docs repo to fix/align automation around (1) syncing labels back to the originating dev PR when a docs PR closes, and (2) creating the next version docs branch after a version branch is merged.
Changes:
- Update
actions/create-github-app-tokenusage to passclient-id(matching the other automation workflows in this repo). - Adjust milestone selection logic to run the JQ filter via standalone
jq(enabling--argusage), used to compute the next docs branch name. - Minor whitespace/blank-line normalization in workflow YAML.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.github/workflows/docs-pr-sync.yml |
Aligns GitHub App token generation input to client-id; whitespace normalization. |
.github/workflows/docs-branch-create.yml |
Aligns token generation input to client-id; changes milestone query to use standalone jq with --arg for version comparison. |
|
Newest code from mattermost has been published to preview environment for Git SHA 3e99f84 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
💤 Files with no reviewable changes (2)
📝 WalkthroughWalkthroughThe pull request updates two documentation GitHub Actions workflows. It changes milestone lookup and branch creation in the branch workflow, updates GitHub App token input naming, and preserves docs PR parsing, label synchronization, and commenting behaviour. ChangesDocumentation workflow updates
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/docs-branch-create.yml:
- Around line 134-163: Update the gh api milestones pipeline to use jq slurp
mode and flatten all paginated response arrays before applying JQ_FILTER,
ensuring milestone sorting and selection occur across every page. Keep
NEXT_TITLE as a single result so VERSION and DOCS_BRANCH derive from the
globally earliest matching milestone.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 35d19c29-4edb-4bd7-b503-42a5323a78d5
📒 Files selected for processing (2)
.github/workflows/docs-branch-create.yml.github/workflows/docs-pr-sync.yml
|
Newest code from mattermost has been published to preview environment for Git SHA e2c6975 |
|
Newest code from mattermost has been published to preview environment for Git SHA b6676e1 |
|
Newest code from mattermost has been published to preview environment for Git SHA a0faae5 |
|
Newest code from mattermost has been published to preview environment for Git SHA c5a38d3 |
Relates to mattermost/mattermost#37401.