Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Modifies latest-allowed to only match stable versions #439

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.

Sign up for GitHub

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

Jump to bottom
Open
vahid-haghighat wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Modifies latest-allowed to only match stable versions #439

vahid-haghighat wants to merge 7 commits into from
+48 −6

Conversation

@vahid-haghighat
Copy link

@vahid-haghighat vahid-haghighat commented Dec 1, 2024 •
edited
Loading

Description

In the current implementation, the latest and latest-allowed options have potentially confusing behaviours:

  • latest-allowed can unexpectedly include pre-release versions
  • latest is limited to stable versions only

This discrepancy can cause issues, particularly in CI/CD pipelines where version predictability is crucial.
This PR is fixing that difference in the behaviours and makes sure the latest-allowed only finds stable versions.

@grillba
Copy link

grillba commented Jan 2, 2025 •
edited
Loading

cant it just fix it so latest-allowed works in the same way as latest? This was fixed in version 1.0.0 for latest

Note I'm not a maintainer on this repo, just chanced upon this same issue

Edit: tenv supports only installing stable releases: tenv terraform install latest-allowed

@kvendingoldo
Copy link

kvendingoldo commented Jan 2, 2025

@grillba you can use latest-pre

@vahid-haghighat
Copy link
Author

vahid-haghighat commented Jan 7, 2025

cant it just fix it so latest-allowed works in the same way as latest? This was fixed in version 1.0.0 for latest

Note I'm not a maintainer on this repo, just chanced upon this same issue

Edit: tenv supports only installing stable releases: tenv terraform install latest-allowed

It is possible to fix the latest-allowed, but I wasn't sure if that was the right move. Maybe some people are using pre-releases with latest-allowed? I would have no idea why, though 😄

@Zordrak
Copy link
Collaborator

Zordrak commented Jul 4, 2025

I think I am inclined to agree. latest-allowed and latest should be common. If latest matches only stable, latest-allowed probably should too.

@vahid-haghighat vahid-haghighat changed the title Adds latest-allowed-stable option Modifies latest-allowed to only match stable versions Dec 28, 2025
@vahid-haghighat vahid-haghighat changed the title Modifies latest-allowed to only match stable versions Modifies latest-allowed to only match stable versions Dec 28, 2025
@vahid-haghighat
Copy link
Author

vahid-haghighat commented Dec 28, 2025

@Zordrak I updated the latest-allowed to match stable versions only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vahid-haghighat @grillba @kvendingoldo @Zordrak