Closed emersonknapp closed 4 years ago
Merging #87 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #87 +/- ##
=======================================
Coverage 66.66% 66.66%
=======================================
Files 4 4
Lines 249 249
=======================================
Hits 166 166
Misses 83 83
Flag | Coverage Δ | |
---|---|---|
#unittests | 66.66% <ø> (ø) |
:arrow_up: |
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 0bcdb77...e315c7f. Read the comment docs.
The line adding the ROS apt sources explicitly specified architectures. Since we are working in an emulated environment, the only architecture we need is the "native"/emulated one, so we don't have to specify.
This fixes the use case for armhf builds that have ROS rosdeps, because the armhf sources were not being added.