Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use current repo for tmt-lint pre-commit #3117

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

LecrisUT
Copy link
Contributor

Solves various chicken-and-egg issues when trying to update the linter. Also solves a problem due to not using src-layout (dependencies and python3 -m are taken from the pre-commit, but because import tmt... is used everywhere, the current source has precedence)

@LecrisUT
Copy link
Contributor Author

LecrisUT commented Jul 29, 2024

Great, need to wait for setuptools to fix their issue Resolved

@LecrisUT LecrisUT mentioned this pull request Jul 29, 2024
4 tasks
@LecrisUT LecrisUT force-pushed the feat/pre-commit-self branch from 0018340 to a5c542c Compare July 29, 2024 14:12
@LecrisUT LecrisUT force-pushed the feat/pre-commit-self branch from a5c542c to 8a4abbb Compare July 29, 2024 15:21
@LecrisUT LecrisUT mentioned this pull request Aug 20, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant