Migrate Flogger from WORKSPACE to Bzlmod and bump bazel version to 7.…

…3.1.

Fixes #387.

RELNOTES=n/a
PiperOrigin-RevId: 682479608

.bazelversion

@@ -1 +1 @@
-7.0.2
+7.3.1

.github/workflows/ci.yml

@@ -27,7 +27,7 @@ jobs:
         uses: actions/cache@88522ab9f39a2ea568f7027eddc7d8d8bc9d59c8
         with:
           path: ~/.cache/bazel/*/*/external
-          key: bazel-${{ hashFiles('WORKSPACE') }}
+          key: bazel-${{ hashFiles('MODULE.bazel') }}
           restore-keys: |
             bazel-
       - name: 'Set up JDK ${{ matrix.java }}'
@@ -37,4 +37,4 @@ jobs:
           distribution: 'zulu'
       - name: 'Test'
         shell: bash
-        run: bazel test --test_output=errors //...
+        run: bazelisk test --lockfile_mode=error --test_output=errors //...

MODULE.bazel

@@ -0,0 +1,21 @@
+# Copyright (C) 2024 The Flogger Authors.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+bazel_dep(name = "rules_java", version = "7.12.1")
+bazel_dep(name = "google_bazel_common")
+git_override(
+    module_name = "google_bazel_common",
+    commit = "e2204d625d97dda88112af0b0a9e56ed7712d15c",
+    remote = "https://github.com/google/bazel-common",
+)

api/BUILD

@@ -81,8 +81,7 @@ $$JAVABASE/bin/javac -d $$TMPDIR -source 8 -target 8 -cp $${STACK_GETTER_COMMON}
     $(location src/main/java/com/google/common/flogger/util/JavaLangAccessStackGetter.java)
 cd $$TMPDIR && $$JAR cf $$OUTPUT com/google/common/flogger/util/*StackGetter*.class
     """,
-    toolchains = ["@openjdk8//:runtime"],
-    tools = ["@openjdk8//:runtime"],
+    toolchains = ["@rules_java//toolchains:jdk_8"],
 )
 
 genrule(
@@ -104,8 +103,7 @@ $$JAVABASE/bin/javac -d $$TMPDIR -source 8 -target 8 -cp $${STACK_GETTER_COMMON}
     $(location src/main/java/com/google/common/flogger/util/StackWalkerStackGetter.java)
 cd $$TMPDIR && $$JAR cf $$OUTPUT com/google/common/flogger/util/*StackGetter*.class
     """,
-    toolchains = ["@openjdk11//:runtime"],
-    tools = ["@openjdk11//:runtime"],
+    toolchains = ["@rules_java//toolchains:remote_jdk11"],
 )
 
 java_library(

log4j2/BUILD

@@ -21,10 +21,9 @@ java_library(
     deps = [
         "//api",
         "//api:system_backend",
-        # TODO(cpovirk): Why do we use a mix of google_bazel_common and "direct" deps?
-        "@com_google_errorprone_error_prone_annotations",
-        "@google_bazel_common//third_party/java/log4j2",
+        "@google_bazel_common//third_party/java/error_prone:annotations",
         "@google_bazel_common//third_party/java/jspecify_annotations",
+        "@google_bazel_common//third_party/java/log4j2",
     ],
 )