Skip to content

style: Improve Formatting and Maintainability#98

Merged
faustbrian merged 7 commits intoArkEcosystemArchive:developfrom
sleepdefic1t:style/improve-formatting-and-maintainability
May 29, 2019
Merged

style: Improve Formatting and Maintainability#98
faustbrian merged 7 commits intoArkEcosystemArchive:developfrom
sleepdefic1t:style/improve-formatting-and-maintainability

Conversation

@sleepdefic1t
Copy link
Contributor

This PR improves code formatting and maintainability.

Specifically, this PR does the following:

  • fixes exit statement in circleci/clang_tidy.sh.
  • removes empty space in circleCI config.
  • fixes dirname statement in ARDUINO_IDE.sh.
  • updates formatting in src/ and test/ classes.
  • increases coverage.
  • updates the change log.

tools used for analysis:

  • Clang-Tidy
  • CodeFactor
  • CodeCov

What kind of change does this PR introduce?

  • Code style update

Does this PR introduce a breaking change?

  • Yes
  • No

Does this PR release a new version?

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the develop branch, not the master branch
  • All tests are passing
  • New/updated tests are included

This PR improves code formatting and maintainability.

Specifically, this PR does the following:

- fixes exit statement in circleci/clang_tidy.sh.
- removes empty space in circleCI config.
- fixes dirname statement in ARDUINO_IDE.sh.
- updates formatting in src/ and test/ classes.
- increases codecoverage.
- updates change log.

tools used for analysis:
- Clang-Tidy
- CodeFactor
- CodeCov
improves tests for:
- AbstractNetwork.
- Crypto_Helpers.
- Transaction.
@ghost ghost added Complexity: Undetermined Needs specialized, in-depth review. Type: Standards The issue reports problems with the compliance of contribution guidelines or code standards. labels May 29, 2019
@codecov-io
Copy link

Codecov Report

Merging #98 into develop will increase coverage by 3.86%.
The diff coverage is 87.81%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #98      +/-   ##
===========================================
+ Coverage    87.28%   91.14%   +3.86%     
===========================================
  Files           27       27              
  Lines          739      836      +97     
===========================================
+ Hits           645      762     +117     
+ Misses          94       74      -20
Impacted Files Coverage Δ
src/include/cpp-crypto/networks/testnet.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/configuration/network.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/networks/mainnet.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/transactions/serializer.h 100% <ø> (ø) ⬆️
src/utils/message.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/networks/devnet.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/configuration/fee.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/transactions/deserializer.h 100% <ø> (ø) ⬆️
src/include/cpp-crypto/transactions/transaction.h 66.66% <0%> (+66.66%) ⬆️
src/include/cpp-crypto/helpers/encoding/hex.h 100% <100%> (+10.71%) ⬆️
... and 19 more

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 a1b0629...2b9c485. Read the comment docs.

@faustbrian faustbrian merged commit 5c80c20 into ArkEcosystemArchive:develop May 29, 2019
@sleepdefic1t sleepdefic1t deleted the style/improve-formatting-and-maintainability branch May 29, 2019 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complexity: Undetermined Needs specialized, in-depth review. Type: Standards The issue reports problems with the compliance of contribution guidelines or code standards.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants