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

Backport 5 ➡️ 3 Disabling flaky depth and thermal camera tests on mac (#405) #696

Merged

Conversation

Crola1702
Copy link
Contributor

Backports #405

Reference build (Thermal Camera and Depth Camera test regressions on Citadel): https://build.osrfoundation.org/job/ignition_rendering-ci-ign-rendering3-homebrew-amd64/74/

Signed-off-by: Cristóbal Arroyo [email protected]

➡️ Back port

Port ign-rendering5 to ign-rendering3

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

FYI: @Blast545

@Crola1702 Crola1702 requested a review from iche033 as a code owner July 29, 2022 14:13
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Jul 29, 2022
@Blast545 Blast545 self-requested a review July 29, 2022 14:15
@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #696 (91193c8) into ign-rendering3 (ae5c491) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           ign-rendering3     #696   +/-   ##
===============================================
  Coverage           75.37%   75.37%           
===============================================
  Files                  98       98           
  Lines                7793     7793           
===============================================
  Hits                 5874     5874           
  Misses               1919     1919           

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 ae5c491...91193c8. Read the comment docs.

@chapulina chapulina added tests Broken or missing tests / testing infra macOS macOS support labels Jul 29, 2022
@Blast545 Blast545 merged commit c84a02d into gazebosim:ign-rendering3 Aug 1, 2022
@Crola1702 Crola1702 deleted the Crola1702/5_to_3_29-07-2022 branch August 16, 2022 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel macOS macOS support tests Broken or missing tests / testing infra
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants