-
Notifications
You must be signed in to change notification settings - Fork 10
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
230.Add decorator functions to pass click options correctly #231
Conversation
- this was needed in order to import fre make scripts successfully when creating pytests
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.
underscore convention is backwards!
@ilaflott Hmmm so when I reverse the underscore convention, I keep getting this:
|
since it changes a module name, you need to change the name in other places where that old name is imported, is my guess. this pattern is absolutely doable, most every other subtool currently has it |
let me note that |
I think I may have done it with I see it done for most of the |
Describe your changes
Issue ticket number and link (if applicable)
Checklist before requesting a review