Skip to content

fix(ci): handle workflows publishing issue - #67

Merged
IlyaKhD merged 2 commits into
mainfrom
khd/fix-publishing
Sep 8, 2026
Merged

fix(ci): handle workflows publishing issue#67
IlyaKhD merged 2 commits into
mainfrom
khd/fix-publishing

Conversation

@IlyaKhD

@IlyaKhD IlyaKhD commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Practical Value

Fixes publishing of workflow files (GITHUB_TOKEN is unable to push changes in these)

Before

  • GITHUB_TOKEN is use
  • HEAD commit message is used

After

  • PAT is used
  • commit message can be overridden

@IlyaKhD
IlyaKhD requested a balanced review from Copilot September 8, 2026 11:03
@IlyaKhD IlyaKhD self-assigned this Sep 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

All reviewed changes are acceptable with no unresolved issues.

Pull request overview

Fixes workflow publishing by using a PAT and supporting custom commit subjects.

Changes:

  • Uses RELEASE_BRANCH_TOKEN for publishing.
  • Adds an optional commit-message override with HEAD-subject fallback.
File summaries
File Description
scripts/publish-actions.sh Applies the optional commit-message override.
.github/workflows/publish.yml Configures PAT authentication and message input.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@IlyaKhD IlyaKhD changed the title Khd/fix publishing fix(ci): handle workflows publishing issue Sep 8, 2026
@IlyaKhD
IlyaKhD merged commit 1c29ef4 into main Sep 8, 2026
1 check passed
@IlyaKhD
IlyaKhD deleted the khd/fix-publishing branch September 8, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants