Skip to content

Update min bazel version #2812

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

Merged
merged 2 commits into from
Sep 3, 2024
Merged

Update min bazel version #2812

merged 2 commits into from
Sep 3, 2024

Conversation

scentini
Copy link
Collaborator

@scentini scentini commented Aug 26, 2024

Bazel 6.4 was released in, Oct 2023, we're well beyond the support window.

I updated straight to 7.3.1 because according to the docs it's fully compatible with 7.0:

Bazel 7.3.1 is a patch LTS release. It is fully backward compatible with Bazel 7.0 and contains selected changes by the Bazel community and Google engineers.

@scentini scentini changed the title Update mim bazel version Update min bazel version Aug 26, 2024
@scentini scentini marked this pull request as ready for review August 26, 2024 15:25
@UebelAndre
Copy link
Collaborator

well beyond the support window

Where is the support window documented? I feel like the right move is to go to 7.0 but if there is some existing guidance on min supported versions then I'd say "fire away" :D

@illicitonion
Copy link
Collaborator

Per #415 (comment) I think it's fine to drop 6.x support.

I'm also ok with us jumping straight to 7.3.1, because none of our users should be forced to be on an older 7.x release (they should be able to upgrade without worry), but think it's also fine if we want to support 7.0.

@scentini scentini added this pull request to the merge queue Sep 3, 2024
Merged via the queue into bazelbuild:main with commit 3601d56 Sep 3, 2024
3 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Sep 3, 2024
Now that the min tested Bazel version is `7.3.1`
(#2812) this flag can
safely be flipped.

Relates to #2082
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.

3 participants