Skip to content

feat(workflows): add Pre-Release-As trailer override to pre-release w…#1565

Merged
WilliamBerryiii merged 3 commits into
mainfrom
feat/prerelease-version-override
May 14, 2026
Merged

feat(workflows): add Pre-Release-As trailer override to pre-release w…#1565
WilliamBerryiii merged 3 commits into
mainfrom
feat/prerelease-version-override

Conversation

@katriendg
Copy link
Copy Markdown
Contributor

Description

Added Pre-Release-As: commit trailer support to the custom pre-release workflow. When the HEAD commit on main contains a Pre-Release-As: X.Y.Z trailer, the workflow uses that value instead of the computed version. This mirrors how release-please honors Release-As: for the stable pipeline, giving maintainers a manual escape hatch to force a specific pre-release version number without rewriting git history.

The override is self-consuming: subsequent pushes without the trailer revert to the standard computed math.

Resolves #1564

Related Issue(s)

Fixes #1564

Type of Change

Select all that apply:

Code & Documentation:

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update

Infrastructure & Configuration:

  • GitHub Actions workflow
  • Linting configuration (markdown, PowerShell, etc.)
  • Security configuration
  • DevContainer configuration
  • Dependency update

AI Artifacts:

  • Reviewed contribution with prompt-builder agent and addressed all feedback
  • Copilot instructions (.github/instructions/*.instructions.md)
  • Copilot prompt (.github/prompts/*.prompt.md)
  • Copilot agent (.github/agents/*.agent.md)
  • Copilot skill (.github/skills/*/SKILL.md)

Other:

  • Script/automation (.ps1, .sh, .py)
  • Other (please describe):

Sample Prompts (for AI Artifact Contributions)

Testing

  • npm run lint:yaml — All 50 workflow files passed YAML linting
  • npm run lint:md — 197 files, 0 errors
  • npm run spell-check — 303 files, 0 issues
  • npm run lint:frontmatter — 0 errors, 0 warnings
  • npm run lint:ps — All PowerShell files passed
  • actionlint .github/workflows/release-prerelease-pr.yml — No errors
  • Manual diff review confirmed the override block is correctly positioned between PRE_VERSION assignment and GITHUB_OUTPUT echo

Checklist

Required Checks

  • Documentation is updated (if applicable)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable)
  • Tests added for new functionality (if applicable)

AI Artifact Contributions

Required Automated Checks

The following validation commands must pass before merging:

  • Markdown linting: npm run lint:md
  • Spell checking: npm run spell-check
  • Frontmatter validation: npm run lint:frontmatter
  • Skill structure validation: npm run validate:skills
  • Link validation: npm run lint:md-links
  • PowerShell analysis: npm run lint:ps
  • Plugin freshness: npm run plugin:generate
  • Docusaurus tests: npm run docs:test

Security Considerations

  • This PR does not contain any sensitive or NDA information
  • Any new dependencies have been reviewed for security issues (N/A — no dependency changes)
  • Security-related scripts follow the principle of least privilege (N/A — no security script changes)

Additional Notes

Post-Merge Steps

After this PR merges, two empty override commits must be pushed to main (requires maintainer access):

  1. Release-As: 3.4.0 — corrects PR chore(main): release hve-core 4.0.0 #1184 (stable release) from 4.0.0 to 3.4.0
  2. Pre-Release-As: 3.3.<N> — corrects PR chore(main): pre-release 4.1.129 #1476 (pre-release) from 4.1.x to 3.3.x

The exact <N> value depends on the commit count at push time. See the implementation plan for details.

@katriendg katriendg requested a review from a team as a code owner May 11, 2026 08:14
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.49%. Comparing base (35e5924) to head (967e1cc).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1565      +/-   ##
==========================================
- Coverage   85.50%   85.49%   -0.01%     
==========================================
  Files          82       82              
  Lines       11805    11805              
==========================================
- Hits        10094    10093       -1     
- Misses       1711     1712       +1     
Flag Coverage Δ
pester 83.65% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@WilliamBerryiii WilliamBerryiii merged commit 310c68c into main May 14, 2026
49 checks passed
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.

feat(workflows): add manual version override support to pre-release workflow

5 participants