Skip to content

Release/trigger v5 tag#38

Merged
JeffreyShran merged 3 commits into
mainfrom
release/trigger-v5-tag
Jun 17, 2026
Merged

Release/trigger v5 tag#38
JeffreyShran merged 3 commits into
mainfrom
release/trigger-v5-tag

Conversation

@matreurai

Copy link
Copy Markdown
Collaborator

The root cause
The release workflow in main.yml derives the next version by reading the latest GitHub release tag and bumping the patch number. Because the repo still had a legacy v1.8.0 TASVS tag, the merge of PR #36 produced v1.8.1 instead of 5.0.0.

Fix 1 — version guard main.yml
The workflow now only increments when the latest tag matches v5.x. Any other case (legacy v1.8.x or no tag) produces v5.0.0. So legacy TASVS tags can no longer drive TCASVS versioning.

Fix 2 — trigger path main.yml
The workflow only ran on changes under 5.0/, docker/, or Makefile. I added .github/workflows/main.yml to the trigger paths so merging this fix actually runs the release job.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the release workflow to prevent legacy TASVS tags from affecting TCASVS versioning, and ensures workflow changes can actually trigger the release pipeline.

Changes:

  • Add .github/workflows/main.yml to push/pull_request path filters so workflow edits trigger CI/release.
  • Guard version derivation so only v5.x.y tags are incremented; otherwise default to v5.0.0.
  • Update sponsor text in index.md (“Bentley” → “Bentley Systems”).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
index.md Updates sponsor description text.
.github/workflows/main.yml Adjusts workflow triggers and adds a v5-only version increment guard for releases.

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

Comment thread .github/workflows/main.yml
@JeffreyShran JeffreyShran merged commit f10ec11 into main Jun 17, 2026
2 checks passed
@JeffreyShran JeffreyShran deleted the release/trigger-v5-tag branch June 17, 2026 11:36
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