Skip to content

Conversation

@kcalvinalvin
Copy link
Contributor

Dependency clap v4.1.4 requires rustc 1.64 or newer.

For future reference, seems like the project requires Rust 2021 Edition so likely that 1.56.0 is required even if clap was removed or the requirements were lowered, the minimum would be around 1.56.0

Dependency clap v4.1.4 requires rustc 1.64 or newer.

For future reference, seems like the project requires Rust 2021
Edition so likely that 1.56.0 is required even if clap was removed
or the requirements were lowered, the minimum would be around
1.56.0
@Davidson-Souza
Copy link
Member

I like the idea of making the MSRV explicit. The main reason why to use 2021 is because of async/await support. This node is mainly async-first, no bare threads are used, but node and electrum server run in parallel as tasks. I'll check the MSRV for async/await. Apart from that, nothing stops us from supporting older versions, I can downgrade clap or the Rust edition if possible.

@kcalvinalvin
Copy link
Contributor Author

Ok. Is there anything specific you'd like me to do for this PR?

@Davidson-Souza
Copy link
Member

I was checking whether async/await keywords are a 2021 addition, looks like it's a 2018's addition[1]. Do you think it's a good idea to downgrade the version, or is 2021 ok?

[1] https://doc.rust-lang.org/std/keyword.await.html

@kcalvinalvin
Copy link
Contributor Author

I was checking whether async/await keywords are a 2021 addition, looks like it's a 2018's addition[1]. Do you think it's a good idea to downgrade the version, or is 2021 ok?

I don't have a preference. For me it's enough that it's stated so that the user can find out why the code isn't compiling. So yeah, 2021 is ok.

@Davidson-Souza
Copy link
Member

Ok. ACK 5926487

@Davidson-Souza Davidson-Souza merged commit e75f56b into vinteumorg:master Feb 3, 2023
jaoleal added a commit to jaoleal/Floresta that referenced this pull request Mar 30, 2025
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