Skip to content

Automate tag bumping, use field in workflow to set when to create new tag#168

Merged
skeet70 merged 4 commits into
mainfrom
add-tag-automation
Jul 21, 2026
Merged

Automate tag bumping, use field in workflow to set when to create new tag#168
skeet70 merged 4 commits into
mainfrom
add-tag-automation

Conversation

@skeet70

@skeet70 skeet70 commented Jul 21, 2026

Copy link
Copy Markdown
Member

On merge move the tag if it's an existing tag (the non-major version case), create the tag if its a new one for that action (major version release). Note in RELEASING that's the workflow.

@skeet70
skeet70 requested a review from a team as a code owner July 21, 2026 15:40
@skeet70
skeet70 requested review from coltfred and removed request for a team July 21, 2026 15:40
@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown

File Coverage Lines
All files 66% 66%
src/main.rs 66% 66%

Minimum allowed coverage is 24%

Generated by 🐒 cobertura-action against f570d26

@skeet70
skeet70 requested review from giarc3 and removed request for coltfred July 21, 2026 15:45
Comment thread .github/workflows/move-tags.yaml
echo "No number to increment in '${RELEASEVERS}'" 1>&2
exit 1
fi
VERSION="${BASH_REMATCH[1]}$(( 10#${BASH_REMATCH[2]} + 1 ))${BASH_REMATCH[3]}"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How come this changed?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Something changed with a new version of gawk or difference between the CI version and local, and it was causing the shellcheck stuff to fail. This way avoids extra tool drift. I need to triple check the 10# bit there.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah, it's to force it to be decimal, if bash/C reads leading zeros it defaults to octal.

@github-actions

Copy link
Copy Markdown

Merging this PR will move these workflow tags:

bump-version-v1
check-docker-version-v0
deploy-v1
docker-v2
get-tsp-ref-v1
rebuild-v0
rust-artifact-v0
rust-cargo-update-v1
rust-ci-v3
rust-daily-v1
rust-release-v1
scala-ci-v2
scala-cve-v1
shell-ci-v0
sql-smelter-v1
start-tsp-v1
typescript-ci-v1
typescript-release-v1

doesn't actually matter but maybe will keep someone from looking into it.
@skeet70
skeet70 merged commit 0249491 into main Jul 21, 2026
13 checks passed
@skeet70
skeet70 deleted the add-tag-automation branch July 21, 2026 18:51
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