Skip to content

Commit

Permalink
remove unnecessary v prefix from version tags (#2186)
Browse files Browse the repository at this point in the history
* remove unnecessary v prefix from version tags

As @bkuhlmann espoused in a recent Boulder Ruby talk on Milestones
(https://alchemists.io/articles/milestones), the `v` prefix is
unnecessary and not Ruby convention.

* add changelog
  • Loading branch information
joelhawksley authored Dec 19, 2024
1 parent 644bfb7 commit c5feaed
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
4 changes: 4 additions & 0 deletions docs/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,10 @@ nav_order: 5

## main

* Do not prefix release tags with `v`, per recommendation from @bkuhlmann.

*Joel Hawksley*

## 3.21.0

* Updates testing docs to include an example of how to use with RSpec.
Expand Down
6 changes: 3 additions & 3 deletions script/release
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ remote_history_is_clean() {
}

tag_exists_on_remote() {
git rev-parse --quiet --verify refs/tags/v$1.$2.$3 > /dev/null
git rev-parse --quiet --verify refs/tags/$1.$2.$3 > /dev/null
}

working_tree_is_clean() {
Expand All @@ -28,7 +28,7 @@ working_tree_is_clean() {
}

create_release_branch() {
git switch -c release-v$1-$2-$3
git switch -c release-$1-$2-$3
}

update_readme() {
Expand Down Expand Up @@ -77,7 +77,7 @@ commit() {
}

push() {
git push origin release-v$1-$2-$3
git push origin release-$1-$2-$3

echo "####################################################"
echo "Now, open a PR with this branch and merge it to main"
Expand Down

0 comments on commit c5feaed

Please sign in to comment.