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

False positive for operator-no-unspaced with Rackspace URL #308

Closed
wulfharth opened this issue Feb 6, 2019 · 3 comments
Closed

False positive for operator-no-unspaced with Rackspace URL #308

wulfharth opened this issue Feb 6, 2019 · 3 comments
Labels
Bug 🐛 Something isn't working PR made 🚀 A Pull Request has been created.

Comments

@wulfharth
Copy link

I'm getting a false positive for operator-no-unspaced with Rackspace URLs. The URLs contain a subdomain consisting of two hashes beginning with digits and separated by a hyphen. I can reproduce the bug with the following minimum test case.

config

{
    "plugins": [ "stylelint-scss" ],
    "rules": { "scss/operator-no-unspaced": true }
}

.scss

div { background-image: url(https://99-0a.x.y.rackcdn.com/z.jpg); }

output

 1:35  ×  Expected single space before "/"   scss/operator-no-unspaced
 1:35  ×  Expected single space after "/"    scss/operator-no-unspaced
 1:36  ×  Expected single space before "/"   scss/operator-no-unspaced
 1:36  ×  Expected single space after "/"    scss/operator-no-unspaced
@kristerkari
Copy link
Collaborator

Thanks! I'll see if I can make a unit test to fail using your example

@kristerkari
Copy link
Collaborator

I opened a pull request with a fix:
#309

@kristerkari kristerkari added the PR made 🚀 A Pull Request has been created. label Feb 6, 2019
@kristerkari
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐛 Something isn't working PR made 🚀 A Pull Request has been created.
Development

No branches or pull requests

2 participants