-
Notifications
You must be signed in to change notification settings - Fork 4
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
Simplify output format #13
Conversation
Remove f-strings from logging statements and replace dict() with {}.
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 code looks fine to me, except that I don't see where the value corresponding to Group Name
is ever set.
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.
Clean up on aisle three!
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.
Makes sense 👍 I did have one question for you.
Co-Authored-By: Nick M. <mcdonnnj@users.noreply.github.com>
🗣 Description
This PR simplifies the format of the output CSV so that it only shows the fields that we currently care about.
While I was here, I updated a couple of things to make
pylint
a little bit less angry (see f537891).💭 Motivation and context
The old output format is no longer needed. The new format is all that is now required. Out with the old, in with the new.
🧪 Testing
I executed the script and verified that the output looks as expected.
✅ Checklist