Skip to content

Add rule-of-five members for Angle#186

Merged
chapulina merged 2 commits intoign-math6from
angle_rule_of_five
Jan 20, 2021
Merged

Add rule-of-five members for Angle#186
chapulina merged 2 commits intoign-math6from
angle_rule_of_five

Conversation

@mjcarroll
Copy link
Copy Markdown

This suppresses some warnings with newer GCC/Clang versions.

Signed-off-by: Michael Carroll michael@openrobotics.org

Signed-off-by: Michael Carroll <michael@openrobotics.org>
@mjcarroll mjcarroll requested a review from scpeters as a code owner January 11, 2021 18:17
@github-actions github-actions bot added 🏢 edifice Ignition Edifice 🏰 citadel Ignition Citadel 📜 blueprint Ignition Blueprint 🔮 dome Ignition Dome labels Jan 11, 2021
Signed-off-by: Michael Carroll <michael@openrobotics.org>
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 11, 2021

Codecov Report

Merging #186 (2b326fa) into ign-math6 (bd1173d) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##           ign-math6     #186      +/-   ##
=============================================
- Coverage      99.20%   99.20%   -0.01%     
=============================================
  Files             63       63              
  Lines           6024     6020       -4     
=============================================
- Hits            5976     5972       -4     
  Misses            48       48              
Impacted Files Coverage Δ
include/ignition/math/Angle.hh 100.00% <ø> (ø)
src/Angle.cc 100.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 bd1173d...2b326fa. Read the comment docs.

@chapulina chapulina merged commit bb60be4 into ign-math6 Jan 20, 2021
@chapulina chapulina deleted the angle_rule_of_five branch January 20, 2021 04:29
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 🏢 edifice Ignition Edifice

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants