Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve updated codecheck issues #173

Merged
merged 1 commit into from
Nov 6, 2020
Merged

Conversation

adlarkin
Copy link
Contributor

@adlarkin adlarkin commented Nov 6, 2020

This PR is meant to resolve new codecheck issues introduced by gazebosim/gz-cmake#117.

From my understanding, the plan is to forward port the fixes here to all other ignition versions once gazebosim/gz-cmake#117 is merged.

Signed-off-by: Ashton Larkin ashton@openrobotics.org

Signed-off-by: Ashton Larkin <ashton@openrobotics.org>
@osrf-triage osrf-triage added this to Inbox in Core development Nov 6, 2020
@github-actions github-actions bot added the 📜 blueprint Ignition Blueprint label Nov 6, 2020
@codecov
Copy link

codecov bot commented Nov 6, 2020

Codecov Report

Merging #173 into ign-rendering2 will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           ign-rendering2     #173      +/-   ##
==================================================
- Coverage           51.39%   51.38%   -0.01%     
==================================================
  Files                 117      117              
  Lines               10088    10086       -2     
==================================================
- Hits                 5185     5183       -2     
  Misses               4903     4903              
Impacted Files Coverage Δ
ogre/src/OgreRenderEngine.cc 67.61% <100.00%> (-0.11%) ⬇️
ogre2/src/Ogre2RenderEngine.cc 80.00% <100.00%> (-0.07%) ⬇️

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 74d4f5d...cfc21e5. Read the comment docs.

Core development automation moved this from Inbox to In review Nov 6, 2020
@chapulina chapulina merged commit 8b1d5b5 into ign-rendering2 Nov 6, 2020
Core development automation moved this from In review to Done Nov 6, 2020
@chapulina chapulina deleted the adlarkin/cpplint_fixes branch November 6, 2020 22:10
@j-rivero j-rivero removed this from Done in Core development May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 blueprint Ignition Blueprint
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants