这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@dorpvom
Copy link
Member

@dorpvom dorpvom commented Sep 19, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Sep 19, 2018

Codecov Report

Merging #159 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
+ Coverage   95.78%   95.79%   +<.01%     
==========================================
  Files         307      309       +2     
  Lines       15278    15326      +48     
==========================================
+ Hits        14634    14681      +47     
- Misses        644      645       +1
Impacted Files Coverage Δ
...plugins/unpacking/intel_hex/test/test_intel_hex.py 100% <100%> (ø)
src/plugins/unpacking/intel_hex/code/intel_hex.py 100% <100%> (ø)
...test/integration/scheduler/test_cycle_with_tags.py 98.11% <0%> (-1.89%) ⬇️

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 bc115aa...5cdaa56. Read the comment docs.

@tbarabosch tbarabosch requested review from tbarabosch and removed request for weidenba September 20, 2018 12:38
@tbarabosch
Copy link
Contributor

Looks fine to me.

src/web_interface/static/bootstrap
src/web_interface/static/jstree*
src/web_interface/static/google_charts
src/web_interface/static/highlight.js
Copy link
Contributor

Choose a reason for hiding this comment

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

I suppose that one is not directly related to IntelHex

@tbarabosch tbarabosch merged commit 4318ad0 into master Sep 20, 2018
@dorpvom dorpvom deleted the intel-hex-unpacker branch September 25, 2018 11:36
0roman pushed a commit that referenced this pull request Oct 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants