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

Forward port 6 to main 2023-07-26 #222

Merged
merged 14 commits into from
Jul 26, 2023
Merged

Forward port 6 to main 2023-07-26 #222

merged 14 commits into from
Jul 26, 2023

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Jul 26, 2023

➡️ Forward port

Port gz-launch6 to main

Branch comparison: https://github.com/gazebosim//compare/<TO_BRANCH>...<FROM_BRANCH>

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

methylDragon and others added 14 commits August 9, 2022 10:30
Signed-off-by: Louise Poubel <[email protected]>

Signed-off-by: Louise Poubel <[email protected]>
* Return a message on asset error

Signed-off-by: Nate Koenig <[email protected]>

* cleanup

Signed-off-by: Nate Koenig <[email protected]>

* remove debug

Signed-off-by: Nate Koenig <[email protected]>

Signed-off-by: Nate Koenig <[email protected]>
Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
* add stop and pause to websocket server

Signed-off-by: Mabel Zhang <[email protected]>

* better comments

Signed-off-by: Mabel Zhang <[email protected]>

* generalize sim pause/stop to a service operation

Signed-off-by: Mabel Zhang <[email protected]>

* Use RequestRaw

Signed-off-by: Nate Koenig <[email protected]>

* Rename service to request

Signed-off-by: Nate Koenig <[email protected]>

* Update to new request signature

Signed-off-by: Nate Koenig <[email protected]>

---------

Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Co-authored-by: Nate Koenig <[email protected]>
Co-authored-by: Nate Koenig <[email protected]>
* Small cleanup fixes

Signed-off-by: Nate Koenig <[email protected]>

* Missing colon

Signed-off-by: Nate Koenig <[email protected]>

* Fix linting

Signed-off-by: Steve Peters <[email protected]>

---------

Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Steve Peters <[email protected]>
Co-authored-by: Steve Peters <[email protected]>
The LICENSE file contained a copy of the stanze
used at the top of source code files, while the
actual license was in the COPYING file. So remove
the stanza and put the actual Apache 2.0 license text
in LICENSE.

Similar to gazebosim/gz-math#521.

Signed-off-by: Steve Peters <[email protected]>
Version v2 of the actions/checkout workflow is deprecated,
so switch to v3.

Part of gazebo-tooling/release-tools#862.

Signed-off-by: Steve Peters <[email protected]>
Merge ign-launch2 ➡️  ign-launch5
Merge ign-launch5 ➡️  gz-launch6
@nkoenig nkoenig changed the base branch from gz-launch6 to main July 26, 2023 12:28
@github-actions github-actions bot added the 🌱 garden Ignition Garden label Jul 26, 2023
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #222 (9a9899a) into main (7e28e78) will not change coverage.
The diff coverage is n/a.

❗ Current head 9a9899a differs from pull request most recent head 3cce40f. Consider uploading reports for the commit 3cce40f to get more accurate results

@@           Coverage Diff           @@
##             main     #222   +/-   ##
=======================================
  Coverage   56.90%   56.90%           
=======================================
  Files           3        3           
  Lines         413      413           
=======================================
  Hits          235      235           
  Misses        178      178           

@mjcarroll mjcarroll merged commit 76b4ba6 into main Jul 26, 2023
9 checks passed
@mjcarroll mjcarroll deleted the 6-to-main-20230726 branch July 26, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants