Skip to content

Add TAGS as range parameter, fixes #488 fixes #1027#1418

Merged
HaoZeke merged 1 commit intoairspeed-velocity:mainfrom
timkpaine:tkp/tags
May 20, 2025
Merged

Add TAGS as range parameter, fixes #488 fixes #1027#1418
HaoZeke merged 1 commit intoairspeed-velocity:mainfrom
timkpaine:tkp/tags

Conversation

@timkpaine
Copy link
Contributor

@timkpaine timkpaine commented Jul 13, 2024

ref: #488
ref: #1027

Adds a new supported range_spec TAGS which will just list the tags of the repo and run on those.

  • Don't just list and sort the tags, make sure they are reverse-chronologically sorted. Right now they are alphabetically sorted (which isn't even version-sorted, so not great)
    Running for this repo I get: ['v0.6.3', 'v0.6.2', 'v0.6.1', 'v0.6.0', 'v0.4.2', 'v0.4.1', 'v0.3.1', 'v0.3', 'v0.3b1', 'v0.2.2', 'v0.2.2rc1', 'v0.2.1', 'v0.2', 'v0.2rc2', 'v0.1.1', 'v0.1', 'v0.1rc3', 'v0.1rc2', 'v0.1rc1', 'v0.5.1', 'v0.5', 'v0.4', 'initial'], which matches the output of git tag -l --sort=taggerdate and so looks good

Copy link
Member

@HaoZeke HaoZeke left a comment

Choose a reason for hiding this comment

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

Minor CI related nit.

@timkpaine timkpaine force-pushed the tkp/tags branch 2 times, most recently from 9f75f8a to d655133 Compare September 13, 2024 19:12
@mattip mattip marked this pull request as ready for review May 20, 2025 16:53
@mattip mattip requested a review from HaoZeke May 20, 2025 16:53
@mattip
Copy link
Contributor

mattip commented May 20, 2025

Rebased off main to get an updated CI, and merked ready for review. I think this is a nice feature.

Copy link
Member

@HaoZeke HaoZeke left a comment

Choose a reason for hiding this comment

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

LGTM, and CI. Thanks @timkpaine and @mattip

@HaoZeke HaoZeke merged commit ec2ccb1 into airspeed-velocity:main May 20, 2025
15 checks passed
@timkpaine timkpaine deleted the tkp/tags branch May 20, 2025 17:48
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