-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Further enhance check_consistent.py
#8604
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
Conversation
hook = repo["hooks"][0] | ||
package_name, package_rev = hook["id"], repo["rev"] | ||
package_specifier = SpecifierSet(f"=={package_rev.removeprefix('v')}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels slightly fragile; not sure if there's a better way
It seems to me that by convention, |
Hmm, interesting. I'm not wedded to having them be all be called I guess I think the most important thing is for us to be consistent in whether we call them |
If we wanted to establish a naming convention that distinguishes these files from the rest of typeshed, but also make clear that these aren't "tests" in the conventional sense, that you can run with |
Or |
Ooh I quite like that (though I'd go for the slightly less verbose |
Add tests to make sure that:
test_cases
are.py
files..py
files intest_cases
have names starting withtest_
..pre-commit.config.yaml
also appear inrequirements-tests.txt
.pre-commit.config.yaml
have the same version specifier as the ones inrequirements-tests.txt
Also rename the
check_same_files()
function tocheck_no_symlinks()
, which is more accurate these days.