Split up build script into ‘check’ and ‘generate’ targets.

It doesn’t make sense to generate files in the pre-commit hook or in the
continuous integration pipeline.

.githooks/pre-commit

@@ -14,4 +14,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-./build.py
+./build.py check

.github/workflows/bazel-test.yaml

@@ -44,7 +44,7 @@ jobs:
           release: false
         if: runner.os == 'Windows'
       - name: Run Bazel tests
-        run: python build.py
+        run: python build.py check
         env:
           USE_BAZEL_VERSION: ${{matrix.bazel}}
           # https://github.com/bazelbuild/bazelisk/issues/88#issuecomment-625178467

build.py

@@ -60,14 +60,24 @@ def wrapper() -> None:
 @target
 def default() -> None:
     """Builds all targets."""
+    generate()
+    check()
+
+@target
+def check() -> None:
+    """Builds and tests the project."""
     buildifier()
     nogo()
-    docs()
     # Test both default toolchain and versioned toolchains.
-    check()
+    test()
     versions()
     ext()
 
+@target
+def generate() -> None:
+    """Generates files to be checked in."""
+    docs()
+
 @target
 def buildifier() -> None:
     """Checks that all BUILD files are formatted correctly."""
@@ -89,17 +99,17 @@ def nogo() -> None:
                     raise ValueError('unwanted Go targets in {file} found')
 
 @target
-def check() -> None:
+def test() -> None:
     """Runs the Bazel tests."""
-    _check()
+    _test()
 
 @target
 def versions() -> None:
     """Runs the Bazel tests for all supported Emacs versions."""
     for version in sorted(_versions()):
-        _check(f'--extra_toolchains=//elisp:emacs_{version}_toolchain')
+        _test(f'--extra_toolchains=//elisp:emacs_{version}_toolchain')
 
-def _check(*args: str) -> None:
+def _test(*args: str) -> None:
     _bazel('test', ['//...'], options=['--test_output=errors'],
            postfix_options=args)