refactor: uncouple gazelle plugin from rules_python itself

Author: rickeylev

BUILD.bazel

@@ -45,7 +45,7 @@ filegroup(
         "version.bzl",
         "//python:distribution",
         "//tools:distribution",
-        "@rules_python_gazelle_plugin//:distribution",
+        ##"@rules_python_gazelle_plugin//:distribution",
     ],
     visibility = [
         "//:__subpackages__",

MODULE.bazel

@@ -227,8 +227,8 @@ bazel_dep(name = "another_module", version = "0", dev_dependency = True)
 # Extra gazelle plugin deps so that WORKSPACE.bzlmod can continue including it for e2e tests.
 # We use `WORKSPACE.bzlmod` because it is impossible to have dev-only local overrides.
 bazel_dep(name = "rules_go", version = "0.41.0", dev_dependency = True, repo_name = "io_bazel_rules_go")
-bazel_dep(name = "rules_python_gazelle_plugin", version = "0", dev_dependency = True)
-bazel_dep(name = "gazelle", version = "0.40.0", dev_dependency = True, repo_name = "bazel_gazelle")
+##bazel_dep(name = "rules_python_gazelle_plugin", version = "0", dev_dependency = True)
+##bazel_dep(name = "gazelle", version = "0.40.0", dev_dependency = True, repo_name = "bazel_gazelle")
 
 internal_dev_deps = use_extension(
     "//python/private:internal_dev_deps.bzl",
@@ -260,10 +260,10 @@ dev_rules_python_config.add_transition_setting(
 
 # Add gazelle plugin so that we can run the gazelle example as an e2e integration
 # test and include the distribution files.
-local_path_override(
-    module_name = "rules_python_gazelle_plugin",
-    path = "gazelle",
-)
+##local_path_override(
+##    module_name = "rules_python_gazelle_plugin",
+##    path = "gazelle",
+##)
 
 local_path_override(
     module_name = "other",

tests/integration/BUILD.bazel

@@ -23,13 +23,13 @@ _WORKSPACE_FLAGS = [
     "--enable_workspace",
 ]
 
-_WORKSPACE_GAZELLE_PLUGIN_FLAGS = [
-    "--override_repository=rules_python_gazelle_plugin=../../../rules_python_gazelle_plugin",
-]
+##_WORKSPACE_GAZELLE_PLUGIN_FLAGS = [
+##    "--override_repository=rules_python_gazelle_plugin=../../../rules_python_gazelle_plugin",
+##]
 
-_GAZELLE_PLUGIN_FLAGS = [
-    "--override_module=rules_python_gazelle_plugin=../../../rules_python_gazelle_plugin",
-]
+##_GAZELLE_PLUGIN_FLAGS = [
+##    "--override_module=rules_python_gazelle_plugin=../../../rules_python_gazelle_plugin",
+##]
 
 default_test_runner(
     name = "workspace_test_runner",
@@ -40,28 +40,28 @@ default_test_runner(
     visibility = ["//visibility:public"],
 )
 
-default_test_runner(
-    name = "workspace_test_runner_gazelle_plugin",
-    bazel_cmds = [
-        "info {}".format(" ".join(_WORKSPACE_FLAGS + _WORKSPACE_GAZELLE_PLUGIN_FLAGS)),
-        "test {} //...".format(" ".join(_WORKSPACE_FLAGS + _WORKSPACE_GAZELLE_PLUGIN_FLAGS)),
-    ],
-    visibility = ["//visibility:public"],
-)
+##default_test_runner(
+##    name = "workspace_test_runner_gazelle_plugin",
+##    bazel_cmds = [
+##        "info {}".format(" ".join(_WORKSPACE_FLAGS + _WORKSPACE_GAZELLE_PLUGIN_FLAGS)),
+##        "test {} //...".format(" ".join(_WORKSPACE_FLAGS + _WORKSPACE_GAZELLE_PLUGIN_FLAGS)),
+##    ],
+##    visibility = ["//visibility:public"],
+##)
 
 default_test_runner(
     name = "test_runner",
     visibility = ["//visibility:public"],
 )
 
-default_test_runner(
-    name = "test_runner_gazelle_plugin",
-    bazel_cmds = [
-        "info {}".format(" ".join(_GAZELLE_PLUGIN_FLAGS)),
-        "test {} //...".format(" ".join(_GAZELLE_PLUGIN_FLAGS)),
-    ],
-    visibility = ["//visibility:public"],
-)
+##default_test_runner(
+##    name = "test_runner_gazelle_plugin",
+##    bazel_cmds = [
+##        "info {}".format(" ".join(_GAZELLE_PLUGIN_FLAGS)),
+##        "test {} //...".format(" ".join(_GAZELLE_PLUGIN_FLAGS)),
+##    ],
+##    visibility = ["//visibility:public"],
+##)
 
 # TODO: add compile_pip_requirements_test_from_external_repo