Skip to content

chore(ci): fix formatting in slack message for rebase bot#2306

Merged
jonathanpwang merged 1 commit into
mainfrom
ci/fix-rebase-slack-format
Dec 7, 2025
Merged

chore(ci): fix formatting in slack message for rebase bot#2306
jonathanpwang merged 1 commit into
mainfrom
ci/fix-rebase-slack-format

Conversation

@jonathanpwang

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 7, 2025 23:10

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.

Pull request overview

This PR fixes the formatting of Slack messages in the rebase bot workflow by implementing pipe-delimited array serialization to preserve spaces in branch names. The workflow automatically rebases feature branches against main, and branch names can include suffixes with spaces like "(no changes)", "(rebase conflict)", or arrows like "→" when showing backup relationships.

  • Changed array serialization to use | as a delimiter instead of space to preserve branch names containing spaces
  • Updated array deserialization to split on | delimiter and added empty string checks
  • Updated comments to accurately describe the new delimiter strategy

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/rebase-feature-branches.yml
@jonathanpwang jonathanpwang merged commit cb40dc5 into main Dec 7, 2025
29 checks passed
@jonathanpwang jonathanpwang deleted the ci/fix-rebase-slack-format branch December 7, 2025 23:34
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.

2 participants