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

Update MSRV to Rust v1.64.0. #252

Merged
merged 1 commit into from
Jul 30, 2024
Merged

Conversation

PiotrSikora
Copy link
Member

Technically, this happened as a result of hashbrown update in #195,
but we didn't catch it at the time.

Add tests using MSRV to prevent this from happening in the future.

Technically, this happened as a result of hashbrown update in proxy-wasm#195,
but we didn't catch it at the time.

Add tests using MSRV to prevent this from happening in the future.

Signed-off-by: Piotr Sikora <[email protected]>
@PiotrSikora PiotrSikora requested a review from mpwarres July 21, 2024 22:41
@PiotrSikora
Copy link
Member Author

Note that we published v0.2.2 with incorrect MSRV (1.61.0 instead of 1.64.0). Luckily, a higher MSRV (incorrectly declared as 1.63.0, and not 1.64.0) is declared in the hashbrown dependency, so this shouldn't be a big issue, so I won't be yanking our release unless there is a flood of user reports about it.

Also, v1.64.0 was released in September 2022, so hopefully people are using more recent versions by now...

.github/workflows/rust.yml Show resolved Hide resolved
@PiotrSikora PiotrSikora merged commit 3f4274e into proxy-wasm:main Jul 30, 2024
21 checks passed
@PiotrSikora PiotrSikora deleted the msrv branch July 30, 2024 22:51
antonengelhardt pushed a commit to antonengelhardt/proxy-wasm-rust-sdk that referenced this pull request Oct 23, 2024
Technically, this happened as a result of hashbrown update in proxy-wasm#195,
but we didn't catch it at the time.

Add tests using MSRV to prevent this from happening in the future.

Signed-off-by: Piotr Sikora <[email protected]>
mswaagman-godaddy pushed a commit to mswaagman-godaddy/proxy-wasm-rust-sdk that referenced this pull request Nov 27, 2024
Technically, this happened as a result of hashbrown update in proxy-wasm#195,
but we didn't catch it at the time.

Add tests using MSRV to prevent this from happening in the future.

Signed-off-by: Piotr Sikora <[email protected]>
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.

2 participants