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

fix to check for existing files #79

Merged
merged 1 commit into from
May 26, 2021

Conversation

joshuaherrera
Copy link
Contributor

Addresses #78

This seemed like the least obtrusive way to address this issue and worked on my linux machine.

Apologies if opening an issue, only to submit a pull request right after, is bad form. I'm new to contributing to open source software 😄

@dominikbraun dominikbraun self-requested a review May 25, 2021 07:04
@dominikbraun
Copy link
Owner

Apologies if opening an issue, only to submit a pull request right after, is bad form. I'm new to contributing to open source software 😄

It really isn't, some projects even require you to do this 😄

@joshuaherrera
Copy link
Contributor Author

It really isn't, some projects even require you to do this

This is good to know, thank you.

@dominikbraun dominikbraun merged commit 5bd9aa4 into dominikbraun:main May 26, 2021
@dominikbraun
Copy link
Owner

This PR is included in release v0.6.0.

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.

2 participants