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

Enable caches for all Bazel commands, not just build-related ones. #56

Merged
merged 1 commit into from
Jan 5, 2025

Conversation

phst
Copy link
Contributor

@phst phst commented Jan 4, 2025

@p0deje
Copy link
Member

p0deje commented Jan 5, 2025

What are the commands that don't inherit from build? I'm reading and the docs and it looks like the common is not needed here because build is sufficient https://bazel.build/run/bazelrc:

  • Every command inherits from common
  • The following commands inherit from (and are more specific than) build: test, run, clean, mobile-install, info, print_action, config, cquery, and aquery
  • coverage, fetch, and vendor inherit from test

@phst
Copy link
Contributor Author

phst commented Jan 5, 2025

Not 100% sure, but I guess something like bazel mod deps would not inherit from build and still make use of the repository cache.

@p0deje
Copy link
Member

p0deje commented Jan 5, 2025

Well, it won't hurt to use common, thanks for PR!

@p0deje p0deje merged commit 451432a into bazel-contrib:main Jan 5, 2025
@phst phst deleted the common branch January 5, 2025 23:12
@phst
Copy link
Contributor Author

phst commented Jan 5, 2025

Thanks!

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.

None yet

2 participants