Skip to content

Commit

Permalink
fix tagging logic
Browse files Browse the repository at this point in the history
  • Loading branch information
LandonTClipp committed Jan 2, 2025
1 parent ec134b7 commit 5698f88
Showing 1 changed file with 10 additions and 9 deletions.
19 changes: 10 additions & 9 deletions tools/cmd/tag.go
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ func (t *Tagger) createTag(repo *git.Repository, version string) error {
return nil
}

func (t *Tagger) largestTagSemver(repo *git.Repository) (*semver.Version, error) {
func (t *Tagger) largestTagSemver(repo *git.Repository, major uint64) (*semver.Version, error) {
largestTag, err := semver.NewVersion("v0.0.0")
if err != nil {
return nil, errors.New(err)
Expand Down Expand Up @@ -118,7 +118,7 @@ func (t *Tagger) largestTagSemver(repo *git.Repository) (*semver.Version, error)
if err != nil {
return errors.New(err)
}
if version.GreaterThan(largestTag) {
if version.GreaterThan(largestTag) && version.Major() == major {
largestTag = version
}
return nil
Expand Down Expand Up @@ -152,7 +152,14 @@ func (t *Tagger) Tag() error {
if err != nil {
return errors.New(err)
}
largestTag, err := t.largestTagSemver(repo)

requestedVersion, err := semver.NewVersion(t.Version)
if err != nil {
logger.Err(err).Str("requested-version", string(t.Version)).Msg("error when constructing semver from version config")
return errors.New(err)
}

largestTag, err := t.largestTagSemver(repo, requestedVersion.Major())
if err != nil {
return err
}
Expand All @@ -165,12 +172,6 @@ func (t *Tagger) Tag() error {

logger.Info().Msg("found largest semver tag")

requestedVersion, err := semver.NewVersion(t.Version)
if err != nil {
logger.Err(err).Str("requested-version", string(t.Version)).Msg("error when constructing semver from version config")
return errors.New(err)
}

logger = logger.With().
Stringer("requested-version", requestedVersion).
Logger()
Expand Down

0 comments on commit 5698f88

Please sign in to comment.