Skip to content
This repository has been archived by the owner on Jan 16, 2023. It is now read-only.

normalize path with public API #116

Merged
merged 1 commit into from
Dec 8, 2017
Merged

normalize path with public API #116

merged 1 commit into from
Dec 8, 2017

Conversation

antonagestam
Copy link
Owner

replaces #102
fixes #101

Using public API's as suggested by @nschlemm
Thanks for the contribution! 🍰

@antonagestam antonagestam merged commit 7c256e0 into master Dec 8, 2017
@antonagestam antonagestam deleted the fix-102 branch December 8, 2017 03:27
@nschlemm
Copy link

nschlemm commented Dec 8, 2017

you're welcome! although copying rather than modifying (or cherrypicking from) the pull request doesn't make me a contributor ;)

but more seriously: are you publishing this as a release soon? That'd be great!

@antonagestam
Copy link
Owner Author

@nschlemm I am sorry about that, but I had a hard time cherrypicking since there were so many other changes on that branch. If you want to we can start using a AUTHORS file and I could add you there. The contribution is much appreciated!

I will try to get a release out soon!

@nschlemm
Copy link

nschlemm commented Dec 8, 2017

no worries!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

use public API please
2 participants