Skip to content

key-utils: make it no_std with a std default feature#1398

Merged
plebhash merged 1 commit into
stratum-mining:mainfrom
GPTechinno:key-utils-no-std
Feb 11, 2025
Merged

key-utils: make it no_std with a std default feature#1398
plebhash merged 1 commit into
stratum-mining:mainfrom
GPTechinno:key-utils-no-std

Conversation

@Georges760

Copy link
Copy Markdown
Contributor

because a new std feature is intriduced, the Semver MINOR should be bumped

@codecov

codecov Bot commented Jan 24, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 11.11111% with 8 lines in your changes missing coverage. Please review.

Project coverage is 18.04%. Comparing base (49c78e3) to head (a8885ec).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
utils/key-utils/src/lib.rs 11.11% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1398      +/-   ##
==========================================
- Coverage   18.06%   18.04%   -0.02%     
==========================================
  Files         127      127              
  Lines        9555     9559       +4     
==========================================
- Hits         1726     1725       -1     
- Misses       7829     7834       +5     
Flag Coverage Δ
binary_codec_sv2-coverage 0.00% <ø> (ø)
binary_sv2-coverage 6.95% <ø> (ø)
bip32_derivation-coverage 0.00% <0.00%> (ø)
buffer_sv2-coverage 24.88% <0.00%> (-0.15%) ⬇️
codec_sv2-coverage 0.02% <ø> (ø)
common_messages_sv2-coverage 0.17% <ø> (ø)
const_sv2-coverage 0.00% <ø> (ø)
error_handling-coverage 0.00% <0.00%> (ø)
framing_sv2-coverage 0.37% <ø> (ø)
jd_client-coverage 0.42% <ø> (ø)
jd_server-coverage 9.02% <ø> (ø)
job_declaration_sv2-coverage 0.00% <ø> (ø)
key-utils-coverage 2.38% <11.11%> (-0.01%) ⬇️
mining-coverage 3.14% <ø> (-0.03%) ⬇️
mining_device-coverage 0.00% <ø> (ø)
mining_proxy_sv2-coverage 0.82% <ø> (ø)
noise_sv2-coverage 5.78% <ø> (ø)
protocols 23.90% <ø> (-0.02%) ⬇️
roles 6.43% <ø> (ø)
roles_logic_sv2-coverage 11.64% <ø> (ø)
sv2_ffi-coverage 0.00% <ø> (ø)
template_distribution_sv2-coverage 0.00% <ø> (ø)
translator_sv2-coverage 9.53% <ø> (ø)
utils 25.04% <11.11%> (-0.09%) ⬇️
v1-coverage 3.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@plebhash plebhash requested a review from Shourya742 January 24, 2025 22:25
@Georges760 Georges760 force-pushed the key-utils-no-std branch 3 times, most recently from 14e4d95 to 010279d Compare February 4, 2025 10:02

@Shourya742 Shourya742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rest looks good, just some questions.

Comment thread utils/key-utils/src/lib.rs
Comment thread utils/key-utils/src/lib.rs
@plebhash plebhash merged commit d53b78b into stratum-mining:main Feb 11, 2025
@Georges760 Georges760 deleted the key-utils-no-std branch February 12, 2025 08:40
plebhash added a commit to plebhash/stratum that referenced this pull request Mar 27, 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.

3 participants