Skip to content

Update actions CI#139

Merged
chapulina merged 24 commits intoign-math6from
chapulina/actions_groups
Aug 3, 2020
Merged

Update actions CI#139
chapulina merged 24 commits intoign-math6from
chapulina/actions_groups

Conversation

@chapulina
Copy link
Copy Markdown
Contributor

Update actions CI to work with gazebo-tooling/action-gz-ci#13.

Also, show the collapsibles from gazebo-tooling/action-gz-ci#14 in action.

Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina added the tests Broken or missing tests / testing infra label Jul 20, 2020
@chapulina chapulina requested a review from scpeters as a code owner July 20, 2020 21:23
@github-actions github-actions bot added 🏰 citadel Ignition Citadel 📜 blueprint Ignition Blueprint 🔮 dome Ignition Dome labels Jul 20, 2020
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 20, 2020

Codecov Report

Merging #139 into ign-math6 will decrease coverage by 0.20%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##           ign-math6     #139      +/-   ##
=============================================
- Coverage      99.50%   99.30%   -0.21%     
=============================================
  Files             59       59              
  Lines           5662     5751      +89     
=============================================
+ Hits            5634     5711      +77     
- Misses            28       40      +12     
Impacted Files Coverage Δ
include/ignition/math/MassMatrix3.hh 99.40% <ø> (-0.01%) ⬇️
src/RollingMean.cc 100.00% <ø> (ø)
include/ignition/math/Filter.hh 100.00% <100.00%> (ø)
include/ignition/math/Pose3.hh 100.00% <100.00%> (ø)
include/ignition/math/Vector2.hh 100.00% <100.00%> (ø)
include/ignition/math/Vector4.hh 95.10% <100.00%> (-4.90%) ⬇️

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 c513cde...d827244. Read the comment docs.

Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina requested a review from mjcarroll July 31, 2020 20:00
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Signed-off-by: Louise Poubel <louise@openrobotics.org>
Copy link
Copy Markdown

@mjcarroll mjcarroll left a comment

Choose a reason for hiding this comment

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

Looks great to me!

@chapulina chapulina merged commit c43f478 into ign-math6 Aug 3, 2020
@chapulina chapulina deleted the chapulina/actions_groups branch August 3, 2020 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📜 blueprint Ignition Blueprint 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome tests Broken or missing tests / testing infra

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants