Skip to content

Conversation

@copybara-service
Copy link

@copybara-service copybara-service bot commented Jul 20, 2024

Clean up Bazel workspace rules

  • Migrated most WORKSPACE deps to MODULE.bazel (will be required in Bazel 9.x).
  • Use .bazelversion to declare Bazel version instead of USE_BAZEL_VERSION
    env variable (makes it easier to run locally with same bazel version).
  • Explicitly set source/target for dagger.android api, and use
    .bazelrc to set default source/target for annotation processors.
  • Removed unused bazel common dependencies in WORKSPACE
    • Removed bazel_skylib
    • Removed google_bazel_common
    • Removed rules_python
    • Removed zlib
  • Removed obsolete tools/bazel.rc config

RELNOTES=N/A

@copybara-service copybara-service bot force-pushed the test_654352167 branch 2 times, most recently from ff8e900 to edd674b Compare July 21, 2024 17:02
@copybara-service copybara-service bot changed the title Upgrade to Bazel 7.2.1 Clean up Bazel workspace rules Jul 21, 2024
@copybara-service copybara-service bot force-pushed the test_654352167 branch 4 times, most recently from 97d1183 to 08b4718 Compare July 22, 2024 04:12
  * Migrated most WORKSPACE deps to MODULE.bazel (will be required in Bazel 9.x).
  * Use `.bazelversion` to declare Bazel version instead of `USE_BAZEL_VERSION`
    env variable (makes it easier to run locally with same bazel version).
  * Explicitly set `source`/`target` for `dagger.android` api, and use
    `.bazelrc` to set default `source`/`target` for annotation processors.
  * Removed unused bazel common dependencies in WORKSPACE
     * Removed `bazel_skylib`
     * Removed `google_bazel_common`
     * Removed `rules_python`
     * Removed `zlib`
  * Removed obsolete `tools/bazel.rc` config

RELNOTES=N/A
PiperOrigin-RevId: 654352167
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants