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 jsonrpsee to 0.19.0 #372

Closed
wants to merge 4 commits into from

Conversation

come-maiz
Copy link
Contributor

Depends on #371.

Part of #370.

This one allows us to get the latest version of ring, so it compiles in risc-v.

After v0.18.0, jsonrpsee::core::Error was removed.

These are just naive changes to get it to compile.

See: https://github.com/paritytech/jsonrpsee/releases/tag/v0.18.0
paritytech/jsonrpsee#1137

This change allows to compile with jsonrpsee v0.19.0
@wiz-a16z
Copy link

wiz-a16z bot commented Sep 12, 2024

Wiz Scan Summary

Scan Module Critical High Medium Low Info Total
IaC Misconfigurations 0 0 0 0 0 0
Vulnerabilities 0 0 0 0 3 3
Sensitive Data 0 0 0 0 0 0
Secrets 0 0 0 0 0 0
Total 0 0 0 0 3 3

View scan details in Wiz

come-maiz added a commit to come-maiz/helios that referenced this pull request Sep 12, 2024
@ncitron
Copy link
Collaborator

ncitron commented Sep 18, 2024

Closing in favor of #374

@ncitron ncitron closed this Sep 18, 2024
ncitron pushed a commit that referenced this pull request Sep 26, 2024
* Update jsonrpsee to 0.18.0

* Update jsonrpsee ErrorObject

After v0.18.0, jsonrpsee::core::Error was removed.

These are just naive changes to get it to compile.

See: https://github.com/paritytech/jsonrpsee/releases/tag/v0.18.0

* Update jsonrpsee to 0.19.0

* jsonrpsee Server::start is infallible according to:
paritytech/jsonrpsee#1137

This change allows to compile with jsonrpsee v0.19.0

* Release the linux riscv64gc version

Depends on #372.

Part of #370.

* Install risc-v version with heliosup.

* cargo fmt
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.

3 participants