From 4e48bbc31cf34b1b4ccd100a787d1204ddb8866b Mon Sep 17 00:00:00 2001
From: Pradyun Gedam <pradyunsg@users.noreply.github.com>
Date: Fri, 14 Oct 2022 11:14:23 +0100
Subject: [PATCH] Move check-manifest to a CI check

This makes local runs of `nox -s lint` quicker while still providing
the relevant protections for checking the MANIFEST file.
---
 .github/workflows/ci.yml | 1 +
 .pre-commit-config.yaml  | 6 ------
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d3c64ee40a3..362f712b1ef 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -70,6 +70,7 @@ jobs:
       - run: pip install nox
       - run: nox -s prepare-release -- 99.9
       - run: nox -s build-release -- 99.9
+      - run: pipx run check-manifest
 
   vendoring:
     name: vendoring
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 366d859dba5..098f3bfe7b0 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -73,12 +73,6 @@ repos:
     exclude: ^news/(.gitignore|.*\.(process|removal|feature|bugfix|vendor|doc|trivial).rst)
     files: ^news/
 
-- repo: https://github.com/mgedmin/check-manifest
-  rev: '0.48'
-  hooks:
-  - id: check-manifest
-    args: [--no-build-isolation]
-
 ci:
   autofix_prs: false
   autoupdate_commit_msg: 'pre-commit autoupdate'