build: only run bazel 6 unit tests with bzlmod

Author: jbedard

.github/workflows/ci.yaml

@@ -130,10 +130,11 @@ jobs:
                           major: 6
                       bzlmod: 1
                       folder: e2e/npm_translate_lock_replace_packages
-                    # Don't run bzlmod tests with Bazel 6 to reduce the size of the test matrix
+                    # Don't run workspace tests with Bazel 6 to reduce the size of the test matrix
+                    # and remove the need for bazel 6 workspace support for local development.
                     - bazel-version:
                           major: 6
-                      bzlmod: 1
+                      bzlmod: 0
                     # Don't run workspace smoke test under bzlmod
                     - bzlmod: 1
                       folder: e2e/workspace
@@ -150,27 +151,39 @@ jobs:
                       folder: e2e/npm_link_package
                     - bzlmod: 1
                       folder: e2e/rules_foo
-                    # gyp_no_install_script is broken in an usual way on 6.5.0
+                    # gyp_no_install_script+patch_from_repo+js_run_devserver are broken in an usual way on 6.5.0
                     # that is not worth investigating as we're dropping Bazel 6 support soon
                     - bazel-version:
                           major: 6
-                      bzlmod: 0
                       folder: e2e/gyp_no_install_script
+                    - bazel-version:
+                          major: 6
+                      folder: e2e/patch_from_repo
+                    # devservers on bazel6+bzlmod also broken in an odd way
+                    - bazel-version:
+                          major: 6
+                      folder: e2e/js_run_devserver
+                    - bazel-version:
+                          major: 6
+                      folder: e2e/webpack_devserver
+                    - bazel-version:
+                          major: 6
+                      folder: e2e/webpack_devserver_esm
                     # @bazel/runfiles seems broken with non-bzlmod + bazel7
                     # https://github.com/bazel-contrib/rules_nodejs/issues/3797
                     - bzlmod: 0
                       folder: e2e/runfiles
                 include:
                     - bazel-version:
                           major: 7
-                          version: 7.1.1
+                          version: 7.3.2
                       bzlmod: 1
                       os: windows
                       config: local
                       folder: e2e/bzlmod
                     - bazel-version:
                           major: 7
-                          version: 7.1.1
+                          version: 7.3.2
                       bzlmod: 1
                       os: macos
                       config: local

MODULE.bazel

@@ -51,7 +51,7 @@ bazel_dep(name = "bazel_skylib_gazelle_plugin", version = "1.7.1", dev_dependenc
 bazel_dep(name = "buildifier_prebuilt", version = "7.3.1", dev_dependency = True)
 bazel_dep(name = "gazelle", version = "0.39.1", dev_dependency = True, repo_name = "bazel_gazelle")
 bazel_dep(name = "rules_go", version = "0.50.1", dev_dependency = True)
-bazel_dep(name = "stardoc", version = "0.7.1", dev_dependency = True, repo_name = "io_bazel_stardoc")
+bazel_dep(name = "stardoc", version = "0.6.2", dev_dependency = True, repo_name = "io_bazel_stardoc")
 
 host = use_extension(
     "@aspect_bazel_lib//lib:extensions.bzl",

WORKSPACE

@@ -200,6 +200,10 @@ load("@io_bazel_stardoc//:deps.bzl", "stardoc_external_deps")
 
 stardoc_external_deps()
 
+load("@bazel_features//:deps.bzl", "bazel_features_deps")
+
+bazel_features_deps()
+
 load("@stardoc_maven//:defs.bzl", stardoc_pinned_maven_install = "pinned_maven_install")
 
 stardoc_pinned_maven_install()

docs/BUILD.bazel

@@ -1,49 +1,85 @@
-load("@aspect_bazel_lib//lib:docs.bzl", "stardoc_with_diff_test", "update_docs")
+load(":docs.bzl", "stardoc_with_diff_test", "update_docs")
 
 stardoc_with_diff_test(
     name = "js_binary",
     bzl_library_target = "//js/private:js_binary",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "js_library",
     bzl_library_target = "//js/private:js_library",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "js_run_binary",
     bzl_library_target = "//js/private:js_run_binary",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "js_run_devserver",
     bzl_library_target = "//js/private:js_run_devserver",
     symbol_names = ["js_run_devserver"],
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "js_info_files",
     bzl_library_target = "//js/private:js_info_files",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "js_image_layer",
     bzl_library_target = "//js/private:js_image_layer",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "npm_package",
     bzl_library_target = "//npm/private:npm_package",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "npm_link_package",
     bzl_library_target = "//npm/private:npm_link_package",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
     name = "npm_link_all_packages",
     bzl_library_target = "//npm/private:npm_translate_lock_generate.docs",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
@@ -52,6 +88,10 @@ stardoc_with_diff_test(
     symbol_names = [
         "npm_import",
     ],
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 stardoc_with_diff_test(
@@ -61,10 +101,18 @@ stardoc_with_diff_test(
         "list_patches",
         "npm_translate_lock",
     ],
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 update_docs(
     name = "update",
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 # Demonstration delivery target for Aspect Workflows.
@@ -73,7 +121,11 @@ sh_binary(
     name = "docs_delivery_only_on_change",
     srcs = ["delivery.sh"],
     data = glob(["*.md"]),
-    tags = ["deliverable"],
+    tags = [
+        "deliverable",
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )
 
 # Demonstration delivery target for Aspect Workflows.
@@ -82,4 +134,8 @@ sh_binary(
     name = "docs_delivery",
     srcs = ["delivery.sh"],
     data = glob(["*.md"]),
+    tags = [
+        "skip-on-bazel6",
+        "skip-on-rbe",
+    ],
 )

docs/docs.bzl

@@ -0,0 +1,18 @@
+"""
+    Wrapper around stardoc_with_diff_test that only runs the test if Bazel 7 or greater is being used.
+"""
+
+load("@aspect_bazel_lib//lib:docs.bzl", _stardoc_with_diff_test = "stardoc_with_diff_test", _update_docs = "update_docs")
+load("@aspect_bazel_lib//lib:utils.bzl", "is_bazel_7_or_greater")
+
+def stardoc_with_diff_test(name, **kwargs):
+    """
+        Wrapper around stardoc_with_diff_test that only runs the test if Bazel 7 or greater is being used.
+    """
+    if is_bazel_7_or_greater():
+        _stardoc_with_diff_test(name, **kwargs)
+    else:
+        # buildifier: disable=print
+        print("WARNING: Skipping stardoc_with_diff_test for %s because it requires Bazel 7 or greater" % name)
+
+update_docs = _update_docs