+
Skip to content

Update readme.md #4

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

Merged
merged 1 commit into from
Feb 3, 2023
Merged

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
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载