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

Make on (url) checks for equality against full and partial (sans Query string) urls #107

Merged
merged 1 commit into from
Sep 16, 2013

Conversation

KevM
Copy link
Collaborator

@KevM KevM commented Sep 16, 2013

Discussed on this gist: https://gist.github.com/KevM/6554377#comment-906673

The last test may be overkill. ⛵

Note: this is a breaking change with the existing on as it does a compare and was passing for sub partial urls.

Discussed on this gist: https://gist.github.com/KevM/6554377#comment-906673

Note: this is a breaking change with the existing on as it does a compare and was passing for sub partial urls.
@lefthandedgoat
Copy link
Owner

Thanks! I will try to get a new build out tonight.

lefthandedgoat added a commit that referenced this pull request Sep 16, 2013
Make on (url) checks for equality against full and partial (sans Query string) urls
@lefthandedgoat lefthandedgoat merged commit ddee79e into lefthandedgoat:master Sep 16, 2013
@KevM KevM deleted the onequality branch September 16, 2013 18:36
@lefthandedgoat
Copy link
Owner

I haven't forgotten, just been busy, sorry.

@KevM
Copy link
Collaborator Author

KevM commented Sep 18, 2013

No worries. I already have a solution to side load Canopy when needed. ⛵

@lefthandedgoat
Copy link
Owner

Now in 0.8.7. Thanks for the contribution and sorry for the delay.

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