Skip to content
This repository was archived by the owner on Jun 6, 2022. It is now read-only.

Fixed: No more useless warnings for undefined non custom selectors #23

Merged
merged 1 commit into from
Jul 1, 2015

Conversation

MoOx
Copy link
Contributor

@MoOx MoOx commented Jun 30, 2015

Also:

  • Changed: simpler message when a custom selector is undefined
  • Changed: warnings now use PostCSS message API
  • Some fixes in the Changelog + simpler formatting
  • Added some references in the Changelog

Prepared as 2.2.0

Close #22

Also:
- Changed: simpler message when a custom selector is undefined
- Changed: warnings now use PostCSS message API
- Some fixes in the Changelog + simpler formatting
- Added some references in the Changelog

Prepared as 2.2.0

Close #22
@MoOx
Copy link
Contributor Author

MoOx commented Jun 30, 2015

Review will be simpler if you use the github trick w=1

https://github.com/postcss/postcss-custom-selectors/pull/23/files?w=1

@MoOx
Copy link
Contributor Author

MoOx commented Jun 30, 2015

I just moved the flag test in the right place (at least, I think that's the right place ^^).

@MoOx
Copy link
Contributor Author

MoOx commented Jun 30, 2015

I guess you can merge it and just release it :)

@yisibl
Copy link
Contributor

yisibl commented Jul 1, 2015

Good job! Thank you every much!

yisibl added a commit that referenced this pull request Jul 1, 2015
Fixed: No more useless warnings for undefined non custom selectors #23
@yisibl yisibl merged commit 0faea80 into master Jul 1, 2015
@MoOx MoOx deleted the fix.warnings branch July 1, 2015 04:17
@MoOx
Copy link
Contributor Author

MoOx commented Jul 1, 2015

You didn't push the release as a git tag ;)

@yisibl
Copy link
Contributor

yisibl commented Jul 1, 2015

Please hold on.

@yisibl
Copy link
Contributor

yisibl commented Jul 2, 2015

@MoOx
Copy link
Contributor Author

MoOx commented Jul 2, 2015

it seems there is still a bug, see #20 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When customSelectors is empty, console gets polluted
2 participants