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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
style: formatting, coverage, and improvements #121
style: formatting, coverage, and improvements #121
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #121 +/- ##
===========================================
+ Coverage 91.44% 91.82% +0.37%
===========================================
Files 33 33
Lines 877 856 -21
===========================================
- Hits 802 786 -16
+ Misses 75 70 -5
Continue to review full report at Codecov.
|
Codacy should ignore Current Codacy analysis with those filtered at : codacy.com/sleepdefic1t/cpp-crypto. This one is good to go 馃憤 |
Passing Codacy here (ignoring changelog): codacy/sleepdefic1t/cpp-crypto. This is good to go otherwise, @faustbrian @supaiku0 @kristjank 馃憤 |
@sleepdefic1t conflicts |
remove empty line. remove unused def. fix PR#
readd lib dir
codecov miss
add invalid-versioned network test
This PR updates style and improves formatting, coverage, and the overall implementation.
These are basically fixes for things showing up in Codacy/Codefactor/Codecov.
I did not attempt to update Transactions-related classes and tests; that's next for updates.
Changelog, I think Codacy needs configured for the format our Organization uses.
I followed as closely to keepachangelog.com as possible,
but Codacy still complains 馃し鈥嶁檪
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
Does this PR release a new version?
The PR fulfills these requirements:
develop
branch, not themaster
branch