-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add pycharm, fix detached run, add utils functions #10
Conversation
- Fix run so it will be run detached - add get_dirs_in_dir, read_file and resport_issue to comeback.utils - get_dirs_in_dir - returns 1st level of dirs in a path - read_file - reads a file and closes the fd - report_issue - template for errors that need reporting - Fix the example dir structure
|
||
def run_plugin(cwd): | ||
platform = utils.get_platform() | ||
if platform == "windows": |
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.
Maybe we should Enum
the platform?
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.
What do you mean?
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.
To use Platforms.WINDOWS
or something instead of strings. (enum)
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.
WIll do
run_plugin
get_dirs_in_dir
,read_file
andreport_issue
to comeback.utilsget_dirs_in_dir
- returns 1st level of dirs in a pathread_file
- reads a file and closes the fdreport_issue
- template for errors that need reportingAdds support for #4
fixes #9