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

Hotfix Master - no-url-protocols, Fix regex for detecting url protocol use #347

Merged

Conversation

benthemonkey
Copy link
Member

Closes #340

DCO 1.1 Signed-off-by: Ben Rothman bensrothman@gmail.com

@DanPurdy
Copy link
Member

This looks good, just need to update the branch and we can merge it in.

@benthemonkey
Copy link
Member Author

rebased. Out of curiosity, is it poor form for me to merge in my own PRs if they have approval, pending branch update?

@bgriffith
Copy link
Member

We don't generally do it. I think it's easier for everyone to keep track with what's getting merged into develop.

@bgriffith
Copy link
Member

Update looks good.

bgriffith added a commit that referenced this pull request Oct 23, 2015
Hotfix Master - no-url-protocols, Fix regex for detecting url protocol use
@bgriffith bgriffith merged commit 04face4 into sasstools:master Oct 23, 2015
donabrams pushed a commit to donabrams/sass-lint that referenced this pull request Nov 19, 2015
…ocol-regex

Hotfix Master - no-url-protocols, Fix regex for detecting url protocol use
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants