Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix windows warnings #199

Merged
merged 4 commits into from
Mar 11, 2024
Merged

Fix windows warnings #199

merged 4 commits into from
Mar 11, 2024

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Sep 13, 2022

No description provided.

Signed-off-by: Nate Koenig <nate@openrobotics.org>
@github-actions github-actions bot added the 🌱 garden Ignition Garden label Sep 13, 2022
@codecov
Copy link

codecov bot commented Sep 13, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.90%. Comparing base (e35e043) to head (41df4bc).
Report is 7 commits behind head on gz-launch6.

Additional details and impacted files
@@             Coverage Diff             @@
##           gz-launch6     #199   +/-   ##
===========================================
  Coverage       56.90%   56.90%           
===========================================
  Files               3        3           
  Lines             413      413           
===========================================
  Hits              235      235           
  Misses            178      178           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI is failing

@azeey
Copy link
Contributor

azeey commented Nov 21, 2022

@nkoenig to get back to this

@azeey azeey added the beta Targeting beta release of upcoming collection label Jul 31, 2023
@azeey azeey removed the beta Targeting beta release of upcoming collection label Aug 21, 2023
@mjcarroll
Copy link
Contributor

@osrf-jenkins retest this please

@nkoenig nkoenig requested a review from ahcorde February 14, 2024 13:07
@nkoenig
Copy link
Contributor Author

nkoenig commented Feb 14, 2024

This doesn't fix all of the windows warnings, but it has reduced them. I think it's okay to get this in.

Signed-off-by: Nate Koenig <natekoenig@gmail.com>
@nkoenig nkoenig merged commit c4f4e4d into gz-launch6 Mar 11, 2024
12 of 13 checks passed
@nkoenig nkoenig deleted the nkoenig/fix_win_warnings branch March 11, 2024 12:29
Crola1702 pushed a commit that referenced this pull request May 23, 2024
* Fix windows warnings

Signed-off-by: Nate Koenig <nate@openrobotics.org>

* Fix more windows warnings

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

* Added string include

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

---------

Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <natekoenig@gmail.com>
Co-authored-by: Nate Koenig <nate@openrobotics.org>
@Crola1702 Crola1702 mentioned this pull request May 23, 2024
8 tasks
Crola1702 added a commit that referenced this pull request May 24, 2024
* Fix windows warnings (#199)

* Fix windows warnings

Signed-off-by: Nate Koenig <nate@openrobotics.org>

* Fix more windows warnings

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

* Added string include

Signed-off-by: Nate Koenig <natekoenig@gmail.com>

---------

Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <natekoenig@gmail.com>
Co-authored-by: Nate Koenig <nate@openrobotics.org>

* Include vector

Signed-off-by: Crola1702 <cristobal.arroyo@ekumenlabs.com>

---------

Signed-off-by: Nate Koenig <nate@openrobotics.org>
Signed-off-by: Nate Koenig <natekoenig@gmail.com>
Signed-off-by: Crola1702 <cristobal.arroyo@ekumenlabs.com>
Co-authored-by: Nate Koenig <natekoenig@gmail.com>
Co-authored-by: Nate Koenig <nate@openrobotics.org>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🌱 garden Ignition Garden
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants