René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"SKIP skips invocation by alias but doesn't skip \"Installing environment\"","articleBody":"### search tried in the issue tracker\r\n\r\nSKIP alias\r\n\r\n### describe your issue\r\n\r\nWhen I run \r\n\r\n```sh\r\nSKIP=pylint-alias2 git commit\r\n```\r\n\r\nthe hook aliased as `pylint-alias2` is properly skipped, however, its environment is still initialized. I would expect that it also skips the lengthy initialization of the environment.\r\n\r\nIndeed, running a single hook ([run.py#153](https://github.com/pre-commit/pre-commit/blob/49f95b9ef391639d09136666be375d99a3dccd04/pre_commit/commands/run.py#153)) checks against `hook.id` and `hook.alias`, however `install_hook_envs` is invoked with only filtering against `hook.id` ([run.py#423](https://github.com/pre-commit/pre-commit/blob/49f95b9ef391639d09136666be375d99a3dccd04/pre_commit/commands/run.py)\r\n\r\nIs this inconsistency deliberate? I couldn't find a use case for it. This quick and dirty patch works for us:\r\n\r\n```diff\r\n-        to_install = [hook for hook in hooks if hook.id not in skips]\r\n+        to_install = [hook for hook in hooks if hook.id not in skips and hook.alias not in skips]\r\n```\r\n\r\nIn our use case Initialization is not needed for everyone (monorepo where some non-python devs also do non-python work) and also very expensive unfortunately (6minutes currently to install already cached pip packages)\r\n\r\n(An even better solution would be if initialization wouldn't happen at all if the hook run would we skipped in general (because of `SKIP` variable or because there is no files changed, however, that would be a but bigger patch I suppose))\r\n\r\n### pre-commit --version\r\n\r\npre-commit \r\n\r\n### .pre-commit-config.yaml\r\n\r\n```yaml\r\nrepos:\r\n  - repo: https://github.com/PyCQA/pylint\r\n    rev: v2.14.5\r\n    hooks:\r\n      - name: pylint-alias1\r\n        alias: pylint-alias1\r\n        id: pylint\r\n        additional_dependencies:\r\n          - arrow\r\n      - name: pylint-alias2\r\n        alias: pylint-alias2\r\n        id: pylint\r\n        additional_dependencies:\r\n          - hdfs\r\n```\r\n\r\n\r\n### ~/.cache/pre-commit/pre-commit.log (if present)\r\n\r\n_No response_","author":{"url":"https://github.com/kmARC","@type":"Person","name":"kmARC"},"datePublished":"2022-08-10T13:49:10.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/2478/pre-commit/issues/2478"}