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

cargo: Bump ring from 0.16 to 0.17 #484

Merged
merged 1 commit into from
Dec 24, 2023
Merged

Conversation

Xunop
Copy link
Contributor

@Xunop Xunop commented Dec 23, 2023

The ring v0.16 has some problems with RISCV64 support, and these were fixed after v0.17.

Checklist

  • I picked the correct source and target branch.
  • I included a new entry to the CHANGELOG.md.
  • I checked cargo clippy and cargo fmt. The CI will fail otherwise anyway.
  • (If applicable) I added tests for this feature or adjusted existing tests.
  • (If applicable) I checked if anything in the wiki needs to be changed.

The `ring` v0.16 has some problems with RISCV64 support, and these were
fixed after v0.17.
Xunop added a commit to Xunop/archriscv-packages that referenced this pull request Dec 23, 2023
Bump `ring` from 0.16 to 0.17. The pr has created to: Nukesor/pueue#484
felixonmars pushed a commit to felixonmars/archriscv-packages that referenced this pull request Dec 23, 2023
Bump `ring` from 0.16 to 0.17. The pr has created to: Nukesor/pueue#484
Copy link

codecov bot commented Dec 24, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3724643) 79.58% compared to head (018fe9c) 79.55%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #484      +/-   ##
==========================================
- Coverage   79.58%   79.55%   -0.04%     
==========================================
  Files          77       77              
  Lines        5521     5521              
==========================================
- Hits         4394     4392       -2     
- Misses       1127     1129       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Test Results

  3 files  +  1   22 suites  +8   2m 50s ⏱️ + 1m 31s
144 tests ±  0  144 ✅ ±  0  0 💤 ±0  0 ❌ ±0 
308 runs  +143  308 ✅ +143  0 💤 ±0  0 ❌ ±0 

Results for commit 018fe9c. ± Comparison against base commit 3724643.

@Nukesor
Copy link
Owner

Nukesor commented Dec 24, 2023

Nice, thanks :)

I'll try to update all other depenendencies and push a point release in the next few days.

@Nukesor Nukesor merged commit 1df7933 into Nukesor:main Dec 24, 2023
19 checks passed
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.

None yet

2 participants