Skip to content
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 support for directories as argument to update command #178 #182

Merged
merged 5 commits into from
Jan 8, 2017

Conversation

jmathai
Copy link
Owner

@jmathai jmathai commented Jan 8, 2017

./elodie.py update --title="test2" /Users/jmathai/Downloads/test
/Users/jmathai/Downloads/test/2015-07-Jul/London/2015-07-23_04-31-12-img_9414-test.jpg -> /Users/jmathai/Downloads/test/2015-07-Jul/London/2015-07-23_04-31-12-img_9414-test2.jpg
{"source":"/Users/jmathai/Downloads/test/2015-07-Jul/London/2015-07-23_04-31-12-img_9414-test.jpg", "destination":"/Users/jmathai/Downloads/test/2015-07-Jul/London/2015-07-23_04-31-12-img_9414-test2.jpg"}
/Users/jmathai/Downloads/test/2016-06-Jun/Ohio/2016-06-21_06-50-36-img_1279-test.mov -> /Users/jmathai/Downloads/test/2016-06-Jun/Ohio/2016-06-21_06-50-36-img_1279-test2.mov
{"source":"/Users/jmathai/Downloads/test/2016-06-Jun/Ohio/2016-06-21_06-50-36-img_1279-test.mov", "destination":"/Users/jmathai/Downloads/test/2016-06-Jun/Ohio/2016-06-21_06-50-36-img_1279-test2.mov"}
****** SUMMARY ******
Metric      Count
--------  -------
Success         2
Error           0

@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Coverage remained the same at 92.683% when pulling bfd6cc4 on directories-for-update-gh-178 into 26c9b85 on master.

@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Coverage remained the same at 92.683% when pulling bfd6cc4 on directories-for-update-gh-178 into 26c9b85 on master.

@jalsin82
Copy link

jalsin82 commented Jan 8, 2017

image

It seems to work as long as you don’t add the trailing “\” .

@jalsin82
Copy link

jalsin82 commented Jan 8, 2017

It works better if I spell the city correctly so that it actually exists in MapQuest.

image

@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Coverage increased (+0.3%) to 93.031% when pulling 5b1fb61 on directories-for-update-gh-178 into 26c9b85 on master.

@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Coverage increased (+0.3%) to 93.031% when pulling 491a582 on directories-for-update-gh-178 into 26c9b85 on master.

@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Coverage increased (+0.3%) to 93.031% when pulling 61d1320 on directories-for-update-gh-178 into 26c9b85 on master.

@jmathai jmathai merged commit 78880a6 into master Jan 8, 2017
@jmathai
Copy link
Owner Author

jmathai commented Jan 8, 2017

Merged.

@jmathai jmathai deleted the directories-for-update-gh-178 branch January 8, 2017 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants