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

[Foxy backport]: Disable a Windows platform warning. (#311) #319

Merged
merged 1 commit into from Dec 3, 2020

Conversation

clalancette
Copy link
Contributor

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

This closes #310 .

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@clalancette
Copy link
Contributor Author

clalancette commented Dec 3, 2020

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette
Copy link
Contributor Author

The Windows jobs are still yellow, but that's because of the (unrelated) CMake warnings. We'd also have to backport some additional stuff for that. The MSBuild warnings that this PR targets are gone.

So I'm going to go ahead and merge this one to Foxy.

@clalancette clalancette merged commit 05c844e into foxy Dec 3, 2020
@delete-merged-branch delete-merged-branch bot deleted the clalancette/foxy-backport-311 branch December 3, 2020 21:03
@jacobperron jacobperron added this to Proposed in Foxy Patch Release 4 via automation Dec 8, 2020
@jacobperron jacobperron moved this from Proposed to Needs release in Foxy Patch Release 4 Dec 8, 2020
@jacobperron jacobperron moved this from Needs release to Released in Foxy Patch Release 4 Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants