Skip to content
This repository has been archived by the owner on Jun 10, 2021. It is now read-only.

Use setup-ros-docker for the e2e test #149

Closed
wants to merge 3 commits into from
Closed

Conversation

emersonknapp
Copy link
Contributor

For consistency with the regular tests, and to get the latest improvements in setup-ros-docker

For consistency with the regular tests, and to get the latest improvements in setup-ros-docker

Signed-off-by: Emerson Knapp <eknapp@amazon.com>
@emersonknapp emersonknapp requested a review from mm318 June 15, 2020 23:02
@codecov
Copy link

codecov bot commented Jun 15, 2020

Codecov Report

Merging #149 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   27.72%   27.72%           
=======================================
  Files          42       42           
  Lines        1847     1847           
  Branches      872      872           
=======================================
  Hits          512      512           
  Misses        534      534           
  Partials      801      801           
Flag Coverage Δ
#unittests 27.72% <ø> (ø)
Impacted Files Coverage Δ
...r/msg/dds_fastrtps/dummy_message__type_support.cpp
...cs_collector/msg/dummy_message__type_support_c.cpp
...etail/dds_fastrtps/dummy_message__type_support.cpp 95.00% <0.00%> (ø)
...ector/msg/detail/dummy_message__type_support_c.cpp 0.00% <0.00%> (ø)

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 952275b...c0d7af9. Read the comment docs.

Signed-off-by: Emerson Knapp <eknapp@amazon.com>
Signed-off-by: Emerson Knapp <eknapp@amazon.com>
@emersonknapp
Copy link
Contributor Author

Closing in favor of #150

@emersonknapp emersonknapp deleted the e2e-use-docker-1 branch August 31, 2020 19:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants