Skip to content
New issue

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

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

Already on GitHub? # to your account

Bump min enumset version to 1.1.0 #4317

Merged
merged 2 commits into from
Nov 20, 2023

Conversation

webmaster128
Copy link
Contributor

Description

The symbol enumset::__internal::EnumSetTypeRepr used in wasmer-types was only made public in 1.1.0:
Lymia/enumset@v1.0.13...v1.1.0

In order to ensure a project embedding Wasmer does not use incompatible enumset versions ^1.0.0, < 1.1.0 we should bump the version here to the actual min required version.

The symbol `enumset::__internal::EnumSetTypeRepr` used in types was only
made public in 1.1.0:
Lymia/enumset@v1.0.13...v1.1.0
Copy link
Member

@syrusakbary syrusakbary left a comment

Choose a reason for hiding this comment

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

Hey @webmaster128 , I think it may be a better idea to set enumset as a workspace dependency, and then just do:

enumset.workspace = true

@webmaster128
Copy link
Contributor Author

Sure.

Do you know what happens to enumset.workspace = true in the package when the crate is published? Is that replaces with the entry from the workspace?

@syrusakbary syrusakbary merged commit 8078169 into wasmerio:master Nov 20, 2023
@webmaster128 webmaster128 deleted the enumset-bump branch November 21, 2023 21:33
webmaster128 added a commit to CosmWasm/cosmwasm that referenced this pull request Jan 15, 2024
Wasmer requires enumset 1.1.0 now
(wasmerio/wasmer#4317)
webmaster128 added a commit to CosmWasm/cosmwasm that referenced this pull request Jan 15, 2024
Wasmer requires enumset 1.1.0 now
(wasmerio/wasmer#4317)
webmaster128 added a commit to CosmWasm/cosmwasm that referenced this pull request Jan 16, 2024
Wasmer requires enumset 1.1.0 now
(wasmerio/wasmer#4317)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants