Skip to content

go.mod: go get github.com/apparentlymart/go-versions@v1.0.2#35181

Merged
apparentlymart merged 1 commit intomainfrom
f-upgrade-go-versions
May 24, 2024
Merged

go.mod: go get github.com/apparentlymart/go-versions@v1.0.2#35181
apparentlymart merged 1 commit intomainfrom
f-upgrade-go-versions

Conversation

@apparentlymart
Copy link
Copy Markdown
Contributor

@apparentlymart apparentlymart commented May 18, 2024

This includes a fix in the handling of conflicting constraints referring to prereleases, like "1.0.0-beta.1, !1.0.0-beta.1".

Previously the library would incorrectly treat that as a positive match, rather than having the negative override the positive as happens in all other cases.

(More info in the upstream PR apparentlymart/go-versions#5 )

This fixes #33452 .


This library appears as a transitive dependency of the remote state backend and legacy modules just due to referring to a main module package that itself uses go-versions.

As far as I can tell, the backends and legacy modules don't actually exercise any of the functionality that would use this package and so I don't expect this change to affect the remote state backends at all. Therefore I'm not asking the backend maintainers to review/test this one.

This includes a fix in the handling of conflicting constraints referring
to prereleases, like "1.0.0-beta.1, !1.0.0-beta.1".

Previously the library would incorrectly treat that as a positive match,
rather than having the negative override the positive as happens in all
other cases.
@apparentlymart apparentlymart requested review from a team as code owners May 18, 2024 00:51
@apparentlymart apparentlymart requested review from a team and removed request for a team May 18, 2024 00:51
@apparentlymart apparentlymart self-assigned this May 18, 2024
@apparentlymart apparentlymart merged commit 91c88f4 into main May 24, 2024
@apparentlymart apparentlymart deleted the f-upgrade-go-versions branch May 24, 2024 17:26
@github-actions
Copy link
Copy Markdown
Contributor

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@github-actions
Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Terraform provider version constraint resolution for NOT prerelease

2 participants