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

chore: bump hashbrown to v0.13 && bump MSRV #250

Closed

Conversation

TennyZhuang
Copy link

Signed-off-by: TennyZhuang [email protected]

@xacrimon
Copy link
Owner

xacrimon commented Jan 20, 2023

This PR increases the MSRV beyond what I'd like to do right now in addition to not updating other MSRV-related datapoints such as the readme. Is that a requirement of the hashbrown bump?

@TennyZhuang TennyZhuang changed the title chore: bump hashbrown to 0.13 chore: bump hashbrown to v0.13 && bump MSRV Jan 23, 2023
@TennyZhuang
Copy link
Author

@xacrimon Yes, hashbrown also bumped their MSRV to Rust 1.61, I guess it's acceptable while the stable version has been bumped to 1.66.

@Razican
Copy link

Razican commented Mar 17, 2023

Hello, Rust 1.68 is now stable, and Rust 1.61 has been out since May 2022. Would it be possible to merge this and release it? We're getting duplicate hashbrown dependency versions due to dashmap in Boa :( it would be nice to get a consistent versioning across the crate.

bors bot pushed a commit to boa-dev/boa that referenced this pull request Mar 17, 2023
…2685)

After this, we are still waiting for `indexmap` & `dashmap` to provide the new `hashbrown` to reduce duplicate dependencies, and for `criterion` to remove `clap` and release a new version. We're also waiting for a new version of `icu_datagen` that bumps the `zip` dependency to avoid a potential vulnerability. Ideally, they would also bump the `simple_logger` dependency, which is pretty outdated. In any case, `simple_logger` still uses an unmaintained `atty` dependency.

Relevant issues:
 - xacrimon/dashmap#250
 - unicode-org/icu4x#3150
 - bheisler/criterion.rs#599
 - borntyping/rust-simple_logger#74
@TennyZhuang
Copy link
Author

Hi @xacrimon, can this PR get merged?

@cuviper
Copy link
Contributor

cuviper commented Jun 24, 2023

FWIW, hashbrown 0.14 is out now, although that further requires Rust 1.64.

@TennyZhuang
Copy link
Author

I'd like to bump it to 0.14 in another PR, but I guess the PR can be merged first.

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

Successfully merging this pull request may close these issues.

4 participants