Allow targets to fully opt out of nogo (#4403)

**What type of PR is this?**

Feature

**What does this PR do? Why is it needed?**

By tagging a target with `"no-nogo"`, `nogo` isn't run at all, not even
for fact generation.

**Which issues(s) does this PR fix?**

Work towards #4327

**Other notes for review**

Author: fmeum

go/nogo.rst

@@ -145,6 +145,10 @@ by default. You could exclude the external repositories from ``nogo`` by using t
 not validated with ``nogo`` by default. See the Bzlmod_ guide for more information
 on how to configure the ``nogo`` scope in this case.
 
+You can prevent ``nogo`` from running for a particular target by adding ``"no-nogo"`` to
+``tags``. This can be useful for generated code, which is often large but not interesting
+for static analysis.
+
 Fixes
 --------------------------------
 

go/private/actions/archive.bzl

@@ -62,14 +62,15 @@ def emit_archive(go, source = None, _recompile_suffix = "", recompile_internal_d
     nogo = go.nogo
 
     # nogo is a FilesToRunProvider and some targets don't have it, some have it but no executable.
-    if nogo != None and nogo.executable != None:
+    if nogo != None and nogo.executable != None and not "no-nogo" in go._ctx.attr.tags:
         out_facts = go.declare_file(go, name = source.name, ext = pre_ext + ".facts")
         out_diagnostics = go.declare_directory(go, name = source.name, ext = pre_ext + "_nogo")
         if validate_nogo(go):
             out_nogo_validation = go.declare_file(go, name = source.name, ext = pre_ext + ".nogo")
         else:
             out_nogo_validation = None
     else:
+        nogo = None
         out_facts = None
         out_diagnostics = None
         out_nogo_validation = None

tests/core/nogo/custom/custom_test.go

@@ -134,15 +134,22 @@ go_library(
 )
 
 go_binary(
-	name = "type_check_fail",
-	srcs = ["type_check_fail.go"],
-	pure = "on",
+    name = "type_check_fail",
+    srcs = ["type_check_fail.go"],
+    pure = "on",
 )
 
 go_library(
     name = "panics",
     srcs = ["panics.go"],
-	importpath = "panics",
+    importpath = "panics",
+)
+
+go_library(
+    name = "panics_no_nogo",
+    srcs = ["panics.go"],
+    importpath = "panics_no_nogo",
+    tags = ["no-nogo"],
 )
 
 -- foofuncname.go --
@@ -609,6 +616,10 @@ func Test(t *testing.T) {
 			includes: []string{
 				"panic: function must not be named ShouldPanic",
 			},
+		}, {
+			desc:        "panics_no_nogo",
+			target:      "//:panics_no_nogo",
+			wantSuccess: true,
 		},
 	} {
 		t.Run(test.desc, func(t *testing.T) {