Skip to content

Escape pipe character in inputs.default#34

Merged
baksetercx merged 3 commits intotrunkfrom
fix/escape-pipe
Feb 9, 2026
Merged

Escape pipe character in inputs.default#34
baksetercx merged 3 commits intotrunkfrom
fix/escape-pipe

Conversation

@baksetercx
Copy link
Copy Markdown
Contributor

No description provided.

@baksetercx baksetercx requested a review from Copilot February 9, 2026 14:01
@baksetercx baksetercx self-assigned this Feb 9, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates markdown generation to avoid breaking tables when an input default contains a pipe (|), and aligns the release pipeline/versioning with the new behavior.

Changes:

  • Escape pipe characters in rendered inputs.*.default values in the generated markdown table.
  • Bump action/package version from 1.2.4 to 1.2.5.
  • Adjust release workflow to tag/push the versioned Docker image during the release step.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
src/Actions.hs Introduces escapePipe and applies it when rendering input default values.
gh-actions-docs.cabal Bumps package version to 1.2.5.
action.yml Updates Docker image tag reference to v1.2.5.
.github/workflows/build-release-test.yaml Changes tagging strategy: removes version tag during build; adds tag/push during release step.

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

@baksetercx baksetercx enabled auto-merge (squash) February 9, 2026 14:04
@baksetercx baksetercx disabled auto-merge February 9, 2026 14:10
@baksetercx baksetercx merged commit 2db2936 into trunk Feb 9, 2026
2 of 3 checks passed
@baksetercx baksetercx deleted the fix/escape-pipe branch February 9, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants