-
Notifications
You must be signed in to change notification settings - Fork 360
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
Fixed: output directory generation not working as expected when --output is passed a directory name instead of filename #45
Conversation
…into fix/output_location
@sumanashrestha , changes are done to meet all the requirements you mentioned in the PR discussion. For writing test, I think, it would be better to write modular tests but right now main logic is executed outside of any function. However, for now it is properly tested manually and is ready to merge. |
if(basename): | ||
return basename | ||
else: | ||
return DEFAULT_OUTOUT_FILENAME |
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.
typo DEFAULT_OUTPUT_FILENAME
Fixes #39
The solved problem:
-o
or--output
was not passed or whenfilename
was passed from CLI, it was working wellIt is solved now
Changes done
The fixed code now uses
os.path
module to extractdirname
andfilename
from the provided args and uses them accordingly.