Skip to content

Conversation

@Ndpnt
Copy link
Contributor

@Ndpnt Ndpnt commented May 26, 2015

Add exclude option to avoid prefixing some selectors

Usage:

var css = '.a {} \nhtml{} \n.b{}'

var prefix = require('postcss-prefix-selector')

var out = postcss().use(prefix({
  prefix: '.some-selector ', // <--- notice the traililng space!
  exclude: ['html', '.b']
})).process(css).css

console.log(out) // only .a selector will be prefixed

Bump version to 1.2.0

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 9d7f3fd on Ndpnt:add-exclude-option into 20f4162 on jonathanong:master.

@jonathanong
Copy link
Contributor

added you as a collab. what's your npm name?

@Ndpnt
Copy link
Contributor Author

Ndpnt commented May 26, 2015

Thanks. My npm name is ndpnt

@jonathanong
Copy link
Contributor

added

@Ndpnt
Copy link
Contributor Author

Ndpnt commented May 27, 2015

Thank you for adding me as a collab :)

@Ndpnt
Copy link
Contributor Author

Ndpnt commented May 28, 2015

Hey !
Just to be sure, you added me as a collaborator to let me merge this PR and publish the new version on npm?
Do you have usual contributing conventions like code reviews, tests, semver, doc…?

@jonathanong
Copy link
Contributor

Denver and update docs :)

@valentin-radulescu-hs
Copy link
Contributor

👍 on getting this merged

valentin-radulescu-hs added a commit that referenced this pull request Sep 29, 2015
@valentin-radulescu-hs valentin-radulescu-hs merged commit cd9cc64 into RadValentin:master Sep 29, 2015
@valentin-radulescu-hs
Copy link
Contributor

@jonathanong Can you add me on npm so I can publish the update? Username is valentin-radulescu-hs

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.

4 participants