Skip to content

Bazel updates: prep for BCR release automation#562

Merged
shameekganguly merged 2 commits intogz-sensors10from
shameek/bazel
Dec 11, 2025
Merged

Bazel updates: prep for BCR release automation#562
shameekganguly merged 2 commits intogz-sensors10from
shameek/bazel

Conversation

@shameekganguly
Copy link
Contributor

🦟 Bug fix

Summary

Few fixes in MODULE.bazel as pre-work to add automation to push new releases to BCR:

  • Remove archive_override for gazebo package deps and use Jetty packages from BCR instead. As a result, bazel CI will use released versions of gz deps, which is consistent with cmake CI.
  • Add compatibility_level to match what is set in BCR
  • Add builidifier linting for consistent formatting of bazel files.
    • Added docstring for gz_sensor_library.bzl
  • Bumped rules_cc to 0.2.14 as indicated in resolved bazel build graph

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

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

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
@shameekganguly
Copy link
Contributor Author

Note: this PR cannot be forward-ported to main automatically since we still want the archive_overrides in MODULE.bazel on that branch. I will open a manual forward port PR once this is merged.

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Dec 10, 2025
@shameekganguly shameekganguly merged commit a243066 into gz-sensors10 Dec 11, 2025
14 checks passed
@shameekganguly shameekganguly deleted the shameek/bazel branch December 11, 2025 18:29
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Dec 11, 2025
shameekganguly added a commit that referenced this pull request Dec 11, 2025
Few fixes in MODULE.bazel as pre-work to add automation to push new releases to BCR:

- Remove archive_override for gazebo package deps and use Jetty packages from BCR instead. As a result, bazel CI will use released versions of gz deps, which is consistent with cmake CI.
- Add compatibility_level to match what is set in BCR
- Add builidifier linting for consistent formatting of bazel files.
- Added docstring for gz_sensor_library.bzl
- Bumped rules_cc to 0.2.14 as indicated in resolved bazel build graph

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
@shameekganguly
Copy link
Contributor Author

Note: this PR cannot be forward-ported to main automatically since we still want the archive_overrides in MODULE.bazel on that branch. I will open a manual forward port PR once this is merged.

Forward port in #565

shameekganguly added a commit that referenced this pull request Dec 12, 2025
Few fixes in MODULE.bazel as pre-work to add automation to push new releases to BCR:

- Add compatibility_level to match what is set in BCR
- Add builidifier linting for consistent formatting of bazel files.
- Added docstring for gz_sensor_library.bzl
- Bumped rules_cc to 0.2.14 as indicated in resolved bazel build graph

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
shameekganguly added a commit that referenced this pull request Dec 12, 2025
)

Few fixes in MODULE.bazel as pre-work to add automation to push new releases to BCR:

- Remove archive_override for gazebo package deps and use Jetty packages from BCR instead. As a result, bazel CI will use released versions of gz deps, which is consistent with cmake CI.
- Add compatibility_level to match what is set in BCR
- Add builidifier linting for consistent formatting of bazel files.
- Added docstring for gz_sensor_library.bzl
- Bumped rules_cc to 0.2.14 as indicated in resolved bazel build graph

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
iche033 pushed a commit that referenced this pull request Dec 13, 2025
) (#566)

Few fixes in MODULE.bazel as pre-work to add automation to push new releases to BCR:

- Remove archive_override for gazebo package deps and use Jetty packages from BCR instead. As a result, bazel CI will use released versions of gz deps, which is consistent with cmake CI.
- Add compatibility_level to match what is set in BCR
- Add builidifier linting for consistent formatting of bazel files.
- Added docstring for gz_sensor_library.bzl
- Bumped rules_cc to 0.2.14 as indicated in resolved bazel build graph

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪵 jetty Gazebo Jetty

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants