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

Return null for URLs not containing a path #7

Merged
merged 1 commit into from
Sep 2, 2016
Merged

Return null for URLs not containing a path #7

merged 1 commit into from
Sep 2, 2016

Conversation

iamwillbar
Copy link
Contributor

When a string that parses successfully as a URL but doesn't contain a path is passed parse would fail with 'Cannot read charAt of null'.

This PR resolves that by returning null if the passed URL doesn't contain a path.

When a string that parses successfully as a URL but doesn't contain a path is passed parse would fail with 'Cannot read charAt of null'.

This PR resolves that by returning null if the passed URL doesn't contain a path.
@jonschlinkert
Copy link
Owner

thanks!

@jonschlinkert jonschlinkert merged commit a8cfd76 into jonschlinkert:master Sep 2, 2016
@jonschlinkert
Copy link
Owner

published to npm and released as 0.3.2. thanks again!

@iamwillbar
Copy link
Contributor Author

@jonschlinkert thanks Jon, not seeing the new version on npm though?

@jonschlinkert
Copy link
Owner

really? Let me double check. it's possible the publish didn't go through and I missed the error message

@jonschlinkert
Copy link
Owner

jonschlinkert commented Sep 6, 2016

you were right! It's now published, sorry about the confusion

edit: fwiw this is a good example of why automation isn't infallible (as long as a human is on one end of it lol)

@iamwillbar
Copy link
Contributor Author

Perfect, I was able to consume the package. Thanks!

@jonschlinkert
Copy link
Owner

thanks for using it, and for creating the issue!

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