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

Conversation

@dorpvom
Copy link
Member

@dorpvom dorpvom commented Apr 3, 2018

When adding ?summary=true to a firmware request, the summaries are collected from all child objects.

@dorpvom dorpvom self-assigned this Apr 3, 2018
@dorpvom dorpvom requested a review from weidenba April 3, 2018 13:21
@codecov-io
Copy link

codecov-io commented Apr 3, 2018

Codecov Report

Merging #90 into master will increase coverage by 0.11%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
+ Coverage   93.95%   94.06%   +0.11%     
==========================================
  Files         284      284              
  Lines       13620    13690      +70     
==========================================
+ Hits        12796    12878      +82     
+ Misses        824      812      -12
Impacted Files Coverage Δ
src/helperFunctions/rest.py 100% <100%> (+12.98%) ⬆️
src/web_interface/rest/rest_firmware.py 96.9% <100%> (+0.16%) ⬆️
...test/unit/web_interface/rest/test_rest_firmware.py 100% <100%> (ø) ⬆️
...tegration/web_interface/rest/test_rest_firmware.py 94.04% <100%> (+0.37%) ⬆️
src/test/unit/helperFunctions/test_rest.py 100% <100%> (ø) ⬆️
src/test/common_helper.py 89.62% <0%> (+0.94%) ⬆️

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 c4efbef...ad26056. Read the comment docs.

@weidenba
Copy link
Contributor

weidenba commented Apr 3, 2018

please add new parameter to wiki

@weidenba weidenba merged commit 277eaf8 into master Apr 3, 2018
@weidenba weidenba deleted the summary-parameter-in-rest-firmware branch April 4, 2018 05:14
weidenba added a commit that referenced this pull request Oct 22, 2018
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.

4 participants