Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot hide Github pre-releases #71

Open
nitrocode opened this issue Nov 20, 2021 · 4 comments
Open

Cannot hide Github pre-releases #71

nitrocode opened this issue Nov 20, 2021 · 4 comments

Comments

@nitrocode
Copy link

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

@sgreben
Copy link
Member

sgreben commented Nov 20, 2021

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
Copy link
Member

sgreben commented Nov 20, 2021

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
Copy link
Member

sgreben commented Nov 20, 2021

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.

@nitrocode
Copy link
Author

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

@nitrocode nitrocode changed the title Cannot hide pre-releases Cannot hide Github pre-releases Dec 7, 2021
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

No branches or pull requests

2 participants