Fix the files pattern in Poetry pre-commit hooks

[floatingpurr]

Pull Request Check List

Resolves: #2511 (comment)

Affected hooks:

• poetry-check
• poetry-export

• Added tests for changed code. not necessary
• Updated documentation for changed code. already present

What a surprise. Poetry-based pre-commit hooks are going to be backed into Poetry itself! 🎉

I've just given them a spin this way:

# export python requirements
- repo: https://github.com/python-poetry/poetry
  rev: 2fa6c17 # put here the next release with the hooks
  hooks:
    - id: poetry-export

but it didn't work. The output is:

$ pre-commit run poetry-export --files poetry.lock
poetry-export........................................(no files to check)Skipped

It looks like poetry.lock doesn't match the files pattern and skips the files.
This quick PR fix this problem.

Tests pointing at my PR with pre-commit 2.16.0 on a *nix enviroment:

repos:
  - repo: https://github.com/floatingpurr/poetry
    rev: 26651c8
    hooks:
      - id: poetry-export

  - repo: https://github.com/floatingpurr/poetry
    rev: 26651c8
    hooks:
      - id: poetry-check

$ pre-commit run poetry-export --files poetry.lock
poetry-export............................................................Passed
$ pre-commit run poetry-check --files pyproject.toml
poetry-check.............................................................Passed

Hope this helps

[Cielquan]

I tested this myself and can confirm the issue and that the fix is working.

As the author of the PR #2511 which introduced this I dunno why this went undetected.

EDIT: pre-commit version 2.13 on linux

[floatingpurr]

Thank you for your check.

[floatingpurr]

Why was black failing on ./src/poetry/utils/env.py?

Edit: probably this PR has been rebased. Thanx

[neersighted]

A quirk of pre-commit -- an earlier PR did not touch src/poetry/utils/env.py, but did update the black/pre-commit version. Because of that, the PR was successfully merged into master but started failing once pre-commit did a run against all (instead of changed) files.

Might be a bug in pre-commit.ci (not sure if it's expected or not), but it's certainly an edge case.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.