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

Create subdirectories if specified in password path #14

Merged
merged 2 commits into from
Jan 17, 2018

Conversation

mdellweg
Copy link
Contributor

@mdellweg mdellweg commented Nov 1, 2017

This patch allows to specify password entries in subdirectories.
If the directory does not exist, it will be created before invoking gnupg.

@aviau
Copy link
Owner

aviau commented Nov 1, 2017

Will take a look soon! Thanks for the pr :)

This patch allows to specify password entries in subdirectories.
If the directory does not exist, it will be created before invoking gnupg.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 90.68% when pulling af7a506 on mdellweg:create_subdirs_if_needed into 7f97083 on aviau:master.

@aviau
Copy link
Owner

aviau commented Jan 11, 2018

Can you assert that the directory does not exist before?

Sorry for the slow reviews! I'll try to review faster.

@mdellweg
Copy link
Contributor Author

Do you mean before creating the path, or in the tests?

@aviau
Copy link
Owner

aviau commented Jan 16, 2018

in the tests.

@mdellweg
Copy link
Contributor Author

Now that travis seems to be functional again, can you restart the test?
I'm sure, it passes ;)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 90.698% when pulling ea3c11d on mdellweg:create_subdirs_if_needed into 7f97083 on aviau:master.

@aviau aviau merged commit f740b1b into aviau:master Jan 17, 2018
@mdellweg mdellweg deleted the create_subdirs_if_needed branch January 17, 2018 19:36
@mdellweg
Copy link
Contributor Author

Thank you! Very nice project btw!

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