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.10.0 #369

Merged
merged 3 commits into from
Jan 26, 2022
Merged

🎈 6.10.0 #369

merged 3 commits into from
Jan 26, 2022

Conversation

chapulina
Copy link
Contributor

🎈 Release

Prepare for 6.10.0 release.

Comparison to 6.9.2: ignition-math6_6.9.2...ign-math6

This follows the 6.9.3~pre2 pre-release:

Various updates have been made to packaging too:

I think we're in a good point to call our Python interfaces a feature, that's why I bumped the minor instead of the patch version.

Required by

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):

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]>
@github-actions github-actions bot added Gazebo 1️1️ Dependency of Gazebo classic version 11 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress 🏰 citadel Ignition Citadel labels Jan 25, 2022
@codecov
Copy link

codecov bot commented Jan 25, 2022

Codecov Report

Merging #369 (b758f70) into ign-math6 (ceab8c7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           ign-math6     #369   +/-   ##
==========================================
  Coverage      99.65%   99.65%           
==========================================
  Files             67       67           
  Lines           6380     6380           
==========================================
  Hits            6358     6358           
  Misses            22       22           

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 ceab8c7...b758f70. Read the comment docs.

@scpeters
Copy link
Member

I just did a quick test of ign-math6 on macOS with osrf/homebrew-simulation#1776 and I'm seeing just one python test failure and several ruby test failures. I think it will be pretty easy to fix the python test failures and enable the pybind11 dependency in homebrew and revisit the ruby failures later

@chapulina
Copy link
Contributor Author

I think it will be pretty easy to fix the python test failures and enable the pybind11 dependency in homebrew and revisit the ruby failures later

Nice! Do you want to tackle the test failure before the release? It would be great to get this release out today or tomorrow, so that we can merge the ign-gazebo PR by the end of the week 🙏

@scpeters
Copy link
Member

I think it will be pretty easy to fix the python test failures and enable the pybind11 dependency in homebrew and revisit the ruby failures later

Nice! Do you want to tackle the test failure before the release? It would be great to get this release out today or tomorrow, so that we can merge the ign-gazebo PR by the end of the week 🙏

I'll open a pull request today to support pybind11

@scpeters
Copy link
Member

I think it will be pretty easy to fix the python test failures and enable the pybind11 dependency in homebrew and revisit the ruby failures later

Nice! Do you want to tackle the test failure before the release? It would be great to get this release out today or tomorrow, so that we can merge the ign-gazebo PR by the end of the week 🙏

I'll open a pull request today to support pybind11

#361 and osrf/homebrew-simulation#1793

@chapulina
Copy link
Contributor Author

#361 and osrf/homebrew-simulation#1793

Thanks, @scpeters . #361 is in and I updated the changelog.

@@ -86,6 +87,7 @@
* [Pull request #317](https://github.com/ignitionrobotics/ign-math/pull/317)
1. Quaternion
* [Pull request #324](https://github.com/ignitionrobotics/ign-math/pull/324)
* [Pull request #361](https://github.com/ignitionrobotics/ign-math/pull/361)
Copy link
Member

Choose a reason for hiding this comment

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

this is in two places?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah because that PR affected both Quaternion and PID

Copy link
Member

@scpeters scpeters left a comment

Choose a reason for hiding this comment

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

ok, we can go ahead and make the release, but let's wait to build bottle until we've merged osrf/homebrew-simulation#1793

@chapulina chapulina merged commit 286a5d4 into ign-math6 Jan 26, 2022
@chapulina chapulina deleted the chapulina/6.10.0 branch January 26, 2022 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress Gazebo 1️1️ Dependency of Gazebo classic version 11
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants