Skip to content

Conversation

@npaun
Copy link
Member

@npaun npaun commented Jan 13, 2026

This PR adds a new flag to our existing dependency types for downloading packages:

  • github_release
  • github_tarball
  • git_clone

If use_bazel_dep: true is set, we generate a bazel_dep() + an override directive (e.g. archive_override() or git_override(). Otherwise, we use the existing templates for a WORKSPACE based repo.

Additionally use_module_bazel_from_bcr: some.version.here can be used to automatically download an inject a MODULE.bazel file from BCR if one is available from there, but not in the project itself.

This is basically the last PR in my project of upgrading update-deps.py. There's probably still room for small simplifications / cleanups down the road, but the vast majority of our dependencies are now automated again.

@npaun npaun requested a review from fhanau January 13, 2026 22:14
@npaun npaun requested review from a team as code owners January 13, 2026 22:14
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 13, 2026

CodSpeed Performance Report

Merging this PR will degrade performance by 84.24%

Comparing npaun/bazel-dep-with-override (59df4a3) with main (b73532d)

Summary

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 157 untouched benchmarks
🆕 2 new benchmarks
⏩ 38 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
Existing_LargeStream_Value 12.3 s 77.7 s -84.24%
🆕 bm_Coro_Fib10 N/A 20 µs N/A
🆕 bm_Promise_Fib10 N/A 19.2 µs N/A
simpleStringBody[Response] 22.1 µs 19 µs +16.36%

Footnotes

  1. 38 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@npaun npaun force-pushed the npaun/bazel-dep-with-override branch from 453cee3 to 59df4a3 Compare January 14, 2026 18:22
@npaun npaun force-pushed the npaun/bazel-dep-with-override branch from 59df4a3 to a8a1239 Compare January 15, 2026 03:40
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.

1 participant