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

Re-disable tests for #202 #438

Merged
merged 1 commit into from
Dec 15, 2020
Merged

Conversation

scpeters
Copy link
Member

This reverts part of "Enable tests for #202 on macOS with Xcode 12+, add catalina workflow (#414)", since those tests have started failing again in the macOS CI action. It keeps the macOS workflow.

I don't know what has changed or why, but I've re-opened #202 and am disabling the tests again here.

This reverts part of commit
0e77816.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Dec 15, 2020
@codecov-io
Copy link

Codecov Report

Merging #438 (f408a90) into sdf9 (f5c8bd9) will increase coverage by 0.15%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             sdf9     #438      +/-   ##
==========================================
+ Coverage   86.26%   86.42%   +0.15%     
==========================================
  Files          59       60       +1     
  Lines        9188     9294     +106     
==========================================
+ Hits         7926     8032     +106     
  Misses       1262     1262              
Impacted Files Coverage Δ
src/Scene.cc 100.00% <100.00%> (ø)
src/Sky.cc 100.00% <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 9995007...f408a90. Read the comment docs.

@scpeters
Copy link
Member Author

merging without review to fix failing tests and unblock other PR's

@scpeters scpeters merged commit 940eb3b into gazebosim:sdf9 Dec 15, 2020
@scpeters scpeters deleted the revert_issue_202_fix branch December 15, 2020 01:22
scpeters added a commit to scpeters/sdformat that referenced this pull request Dec 15, 2020
)

This reverts part of commit
0e77816.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
scpeters added a commit to scpeters/sdformat that referenced this pull request Dec 15, 2020
)

This reverts part of commit
0e77816.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
scpeters added a commit that referenced this pull request Dec 15, 2020
This reverts part of commit
0e77816.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@azeey azeey mentioned this pull request Dec 24, 2020
chapulina pushed a commit that referenced this pull request Dec 24, 2020
This reverts part of commit
0e77816.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
azeey pushed a commit that referenced this pull request Dec 24, 2020
This reverts part of commit
0e77816.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants