Skip to content

Conversation

@EUdds
Copy link
Member

@EUdds EUdds commented Feb 20, 2021

Description

Please include a summary of the change and which issue (if applicable) is fixed

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have formatted my code with ./deploy.sh format
  • My changes generate no new warnings or errors
  • I have added tests/examples that prove my fix is effective or that my feature works
  • New and existing tests/examples pass with my changes

@EUdds EUdds requested a review from bennyE31 February 20, 2021 23:27
@codecov
Copy link

codecov bot commented Feb 20, 2021

Codecov Report

Merging #138 (893e034) into master (187c1d6) will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #138      +/-   ##
=========================================
+ Coverage    2.35%   2.44%   +0.09%     
=========================================
  Files          57      57              
  Lines        3999    4002       +3     
  Branches      734     734              
=========================================
+ Hits           94      98       +4     
+ Misses       3905    3904       -1     
Impacted Files Coverage Δ
...est/run_profiles/Test_load_software_parameters.cpp 100.00% <100.00%> (ø)
embedded/app/src/load_software_parameters.c 100.00% <0.00%> (+3.44%) ⬆️

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 187c1d6...893e034. Read the comment docs.

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.

2 participants