-
-
Notifications
You must be signed in to change notification settings - Fork 8.4k
[bazel] Bump to Bazel 8 #15512
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
base: trunk
Are you sure you want to change the base?
[bazel] Bump to Bazel 8 #15512
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
1f1d49f
to
7c70f0e
Compare
import time | ||
from test.selenium.webdriver.common.network import get_lan_ip | ||
from test.selenium.webdriver.common.webserver import SimpleWebServer | ||
from urllib.request import urlopen | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These 2 imports are are duplicated below:
from test.selenium.webdriver.common.network import get_lan_ip
from test.selenium.webdriver.common.webserver import SimpleWebServer
These 4 are not used:
import socket
import subprocess
import time
from urllib.request import urlopen
PR Type
Enhancement, Configuration changes
Description
Updated Bazel version to 8.1.1 in
.bazelversion
.Adjusted
.bazelrc
configurations for compatibility with Bazel 8.Updated dependencies in
MODULE.bazel
to newer versions.Changes walkthrough 📝
.bazelrc
Update `.bazelrc` for Bazel 8 compatibility
.bazelrc
.bazelversion
Bump Bazel version to 8.1.1
.bazelversion
MODULE.bazel
Update `rules_jvm_external` dependency version
MODULE.bazel
rules_jvm_external
dependency to version 6.7.