terraform-module-versions icon indicating copy to clipboard operation
terraform-module-versions copied to clipboard

Cannot hide Github pre-releases

Open nitrocode opened this issue 3 years ago • 4 comments

My terraform

module "ecs_alb_service_task" {
  source  = "cloudposse/ecs-alb-service-task/aws"
  version = "0.56.0"
}

This shows pre-release for some reason

✗ terraform-module-versions check -pre-release=false .
| UPDATE? |         NAME         | CONSTRAINT | VERSION | LATEST MATCHING | LATEST |
|---------|----------------------|------------|---------|-----------------|--------|
| (Y)     | ecs_alb_service_task | 0.56.0     | 0.56.0  |                 | 0.58.0 |

https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/releases

nitrocode avatar Nov 20 '21 06:11 nitrocode

do you mean 0.58.0? 🤔 a pre-release version would be something like 0.58.0-rc1 (per semver spec, https://semver.org/#spec-item-9)

sgreben avatar Nov 20 '21 06:11 sgreben

ah, i see now, the github release item is marked "pre release". That release should use a standard pre-release tag (0.58.0-something) in this case. This tool only looks at git tags, not github releases (as does terraform itself).

sgreben avatar Nov 20 '21 06:11 sgreben

That said, I wouldn't be opposed to adding a GitHub API check for this (opt-in, since UX-wise it's somewhat suprising to have this special thing for GH). I think GitHub was wrong to introduce a pre-release flag that's independent of tag semver semantics, but that's just something we have to live with.

sgreben avatar Nov 20 '21 22:11 sgreben

Yep, I meant if a github release is marked as a pre-release versus the semver equivalent.

nitrocode avatar Dec 07 '21 00:12 nitrocode