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

remove magic strings #1110

Merged
merged 3 commits into from
Jul 11, 2021
Merged

Conversation

awadhana
Copy link
Contributor

@awadhana awadhana commented Jul 6, 2021

Fixes #1104

Signed-off-by: Hana Awad awadhana0825@gmail.com

Copy link
Member

@petermetz petermetz left a comment

Choose a reason for hiding this comment

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

LGTM once the linter changes are addressed.

@awadhana awadhana force-pushed the fix-pki-magic-strings branch 2 times, most recently from 9c734a3 to e79048d Compare July 8, 2021 16:06
and linter warnings

Fixes hyperledger#1104

Signed-off-by: Hana Awad <awadhana0825@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented Jul 8, 2021

Codecov Report

Merging #1110 (f80c8e8) into main (1ccb9bb) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head f80c8e8 differs from pull request most recent head cd1c0c6. Consider uploading reports for the commit cd1c0c6 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1110      +/-   ##
==========================================
- Coverage   73.15%   73.13%   -0.03%     
==========================================
  Files         245      245              
  Lines        8613     8613              
  Branches      995      995              
==========================================
- Hits         6301     6299       -2     
- Misses       1781     1783       +2     
  Partials      531      531              
Impacted Files Coverage Δ
...main/typescript/rustc-container/rustc-container.ts 66.29% <0.00%> (-2.25%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ccb9bb...cd1c0c6. Read the comment docs.

@petermetz petermetz merged commit 6d9ae53 into hyperledger:main Jul 11, 2021
@awadhana awadhana deleted the fix-pki-magic-strings branch July 15, 2021 04:50
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.

test: fix PKI magic strings in besu tests - genesis account
4 participants