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

🎈 6.5.0 #453

Merged
merged 2 commits into from
Aug 3, 2022
Merged

🎈 6.5.0 #453

merged 2 commits into from
Aug 3, 2022

Conversation

chapulina
Copy link
Contributor

🎈 Release

Preparation for 6.5.0 release.

Comparison to 6.4.0: ignition-gui6_6.4.0...ign-gui6

Needed by:

  • upcoming gz-sim forward-port

Checklist

  • Asked team if this is a good time for a release
  • There are no changes to be ported from the previous major version
  • No PRs targeted at this major version are close to getting in
  • Bumped minor for new features, patch for bug fixes
  • Updated changelog
  • Updated migration guide (as needed)~~
  • Link to PR updating dependency versions in appropriate repository in ignition-release (as needed): Depend on gz-cmake 2.14 gazebo-release/gz-gui6-release#6

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.


Signed-off-by: Louise Poubel <[email protected]>
@chapulina chapulina requested a review from mayman99 August 3, 2022 00:35
@chapulina chapulina requested a review from jennuine as a code owner August 3, 2022 00:35
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Aug 3, 2022
@codecov
Copy link

codecov bot commented Aug 3, 2022

Codecov Report

Merging #453 (3f907e5) into ign-gui6 (011624a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           ign-gui6     #453   +/-   ##
=========================================
  Coverage     66.76%   66.76%           
=========================================
  Files            39       39           
  Lines          5247     5247           
=========================================
  Hits           3503     3503           
  Misses         1744     1744           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link

@quarkytale quarkytale left a comment

Choose a reason for hiding this comment

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

nit but is it okay for some headlines in changelog not start with upper case letter?
CI failing: INTEGRATION_minimal_scene timeouts in focal and INTEGRATION_transport_scene_manager segfaults in jammy. Not sure if these are unrelated

Signed-off-by: Louise Poubel <[email protected]>
@chapulina
Copy link
Contributor Author

nit but is it okay for some headlines in changelog not start with upper case letter?

Good call, I cleaned up the changelog in 3f907e5

CI failing: INTEGRATION_minimal_scene timeouts in focal and INTEGRATION_transport_scene_manager segfaults in jammy. Not sure if these are unrelated

I believe they're flaky. This PR isn't touching any code, so it couldn't be causing new failures.

@chapulina chapulina merged commit 0fb607b into ign-gui6 Aug 3, 2022
@chapulina chapulina deleted the chapulina/6.5.0 branch August 3, 2022 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants