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

Allow lazy multipliers #108

Merged
merged 6 commits into from
May 21, 2024
Merged

Allow lazy multipliers #108

merged 6 commits into from
May 21, 2024

Conversation

qntm
Copy link
Owner

@qntm qntm commented May 21, 2024

For #107.

  • Couldn't get the custom Pylint extension to load, so scrapped it.
  • Couldn't figure out why this utterly bizarre custom Pylint init-hook wasn't working locally, so scrapped it.
  • Seems like despite constraints.txt, new versions of all of these packages have changed the way Pylint works, so scrapped the constraints file.
  • Some Pylint extensions seem to now be core rules, so altered the configuration accordingly.
  • Fixed some linting issues. Notably, added setuptools to requirements.dev.txt.
  • Added unit tests for lazy quantifiers and updated the parser to make the tests pass.
  • Documentation.

@qntm qntm self-assigned this May 21, 2024
@qntm qntm merged commit 408a944 into main May 21, 2024
1 check passed
@qntm qntm deleted the lazy branch May 21, 2024 23:18
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