-
Notifications
You must be signed in to change notification settings - Fork 9
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
Make CI checks required #398
Comments
I just checked the boxes to indicate the current configuration of ign-physics |
Thanks, I thought I had done it for all of them, but the GitHub UI kept acting wreird |
I think ign-common |
Good with me. |
I removed the
|
I just did another pass over all the checks for all libraries and updated their status. I also made sure that Bionic checks are only required up to Fortress, and Jammy checks only from Fortress. |
Closed via gazebosim/gz-common#416 |
This issue keeps track of which CI checks are marked as required for each branch on each library.
Only build checks are listed here, because checks like DCO are already required for all libraries.
Required checks are those which must be green before merging a PR. The ultimate goal is to have all checks required, but we need to address known flakes and failures before requiring them.
This list is expected to be dynamic, and we may have regressions that force us to unrequire checks when new issues are introduced. For example, upstream changes in homebrew introduced warnings into our
ign-common
macOS builds, see gazebosim/gz-common#203.This list only covers release branches that are part of Ignition collections, so it does not cover branches that are still supported because they're Gazebo 9 dependencies.
ign-cmake
All checked
main
ign-common
All checked
ign-common3
ign-common[4-9]
gz-common[0-9]
main
ign-fuel-tools
ign-fuel-tools4
ign-fuel-tools[5-9]
gz-fuel-tools[9]
main
ign-gazebo
ign-gazebo[0-9]
main
ign-gui
ign-gui[0-9]
main
ign-launch
ign-launch[0-4]
ign-launch[5-9]
main
ign-math
All checked
main
ign-msgs
All checked
ign-msgs[0-1]
ign-msgs[2-5]
ign-msgs[6-9]
main
ign-physics
ign-physics[0-4]
ign-physics[5-9]
main
ign-plugin
All checked
main
ign-rendering
ign-rendering[0-5]
ign-rendering[6-9]
main
ign-sensors
ign-sensors[0-5]
ign-sensors[6-9]
main
ign-tools
All checked
main
ign-transport
ign-transport[8-9]
ign-transport[0-9][0-9]
main
ign-utils
All checked
main
sdformat
All checked
sdf[0-9]
sdf[0-9][0-9]
main
The text was updated successfully, but these errors were encountered: