fix(ci): surface skipped clients in sync workflow; bump to Node 24 ac…#7196
Merged
Conversation
…tions Treat parse failures (no version headings) as fatal so upstream CHANGELOG format changes don't silently leave a client stale. Missing CHANGELOG files remain non-fatal (annotated only) so a newly-added client without a published CHANGELOG can't break the nightly. Surface per-client status in the PR body and assign the Copilot coding agent when something needs attention. Bump checkout/setup-node and peter-evans/create-pull-request to versions running on Node 24, and move the script's setup-node input from Node 20 to 22 LTS.
jstirnaman
requested review from
Copilot and
sanderson
and removed request for
a team and
sanderson
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the client-library release notes sync automation to better surface per-client outcomes in CI/PRs, fail fast on parsing drift, and modernize the workflow’s Node/actions versions.
Changes:
- Add per-client status aggregation (table summary), GitHub Actions annotations, and step outputs (
needs_attention,summary) to the sync script. - Update workflow to use newer
checkout,setup-node, andcreate-pull-request, generate a richer PR body from the sync outputs, and conditionally assign Copilot when attention is needed. - Treat “no version headings parsed” as a fatal condition (while missing CHANGELOG remains non-fatal).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| helper-scripts/client-libraries/sync-release-notes.js | Adds result summarization, step summary/output emission, and updated fatal/attention status handling. |
| .github/workflows/sync-client-library-release-notes.yml | Bumps actions/Node versions and composes PR body from sync outputs with conditional assignee. |
Agent-Logs-Url: https://github.com/influxdata/docs-v2/sessions/4825ef09-1133-406c-87ae-af40af3129ec Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
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.
…tions
Treat parse failures (no version headings) as fatal so upstream CHANGELOG format changes don't silently leave a client stale. Missing CHANGELOG files remain non-fatal (annotated only) so a newly-added client without a published CHANGELOG can't break the nightly.
Surface per-client status in the PR body and assign the Copilot coding agent when something needs attention. Bump checkout/setup-node and peter-evans/create-pull-request to versions running on Node 24, and move the script's setup-node input from Node 20 to 22 LTS.