-
Notifications
You must be signed in to change notification settings - Fork 9
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
Try using pre-commit framework #79
Conversation
Just tried. getting error: command not found: pre-commit even after poetry shell and install |
Hmmm, could you try in a clean poetry environment? I don't see why it wouldn't be available if it's installed and the env is activated properly |
just created 2 new poetry envs in PyCharm using Python 3.8 then 3.7. Poetry shell activated. Still getting error |
Could you share your full terminal logs? Including running |
and you've definitely checked out this branch? |
Was in Tuplet branch. Sorry. Just finishing a new algo for text centring. Will complete than, then push. then check out this branch and try again 👍 |
Works! Installing dependencies from lock file Package operations: 19 installs, 2 updates, 0 removals • Updating typing-extensions (4.3.0 -> 3.10.0.2) Installing the current project: neoscore (0.1.7) |
The unimport maintainer suggested this may help with flaky environment issues.
It's also a useful change since it simplifies hook setups and makes them work nicer on Windows.
@craigvear could you try this out? Check out the branch and run: