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

remove autopep8 dependency #1130

Open
edublancas opened this issue Aug 3, 2023 · 0 comments
Open

remove autopep8 dependency #1130

edublancas opened this issue Aug 3, 2023 · 0 comments

Comments

@edublancas
Copy link
Contributor

we use autopep8 to normalize code so whitespace changes do not mark task as outdated:

code = autopep8.fix_code(code)

however, these requires us to add autopep8 as a dependency. we don't really need to use autopep8, as long as we have a preprocessor that normalizes code, it should work. one approach we can take is to strip whitespace in every line. autopep8 does more than this but I think it's worth switching. are there any common cases I'm ignoring?

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

No branches or pull requests

1 participant