-
Notifications
You must be signed in to change notification settings - Fork 45
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
Adding scripts to the checkings #147
Adding scripts to the checkings #147
Conversation
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.
Thanks again! This looks good. Just a minor comment. Feel free to squash and merge yourself.
import csv | ||
from collections import namedtuple |
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.
I would prefer if these stayed in alphabetical order.
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.
pylint or pycodestyle was complaining about this in python 3.9 if I'm not mistaken...
"Only those with write access to this repository can merge pull requests." |
… into Adding_scripts_to_the_checkings
Thanks! |
As mentioned in the other PR, I added the scripts to be checked as well + fixed the issues that came up during these checks.