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

Replace uniq by uniqs #222

Open
wants to merge 1 commit into
base: master
from
Open

Replace uniq by uniqs #222

wants to merge 1 commit into from

Conversation

@A-312
Copy link

A-312 commented Mar 30, 2020

Because this one is used by:

  • postcss-minify-params
  • postcss-unique-selectors

This commit will reduce of one the dependencies number of postcss

@coveralls
Copy link

coveralls commented Mar 30, 2020

Coverage Status

Coverage remained the same at 95.38% when pulling d641743 on A-312:less-dep into 56156f7 on postcss:master.

Because this one is used by:
  - postcss-minify-params
  - postcss-unique-selectors
This commit will reduce of one the dependebcies number of postcss
@A-312 A-312 marked this pull request as ready for review Mar 30, 2020
@evilebottnawi
Copy link
Collaborator

evilebottnawi commented Mar 31, 2020

Why we need it?

@A-312
Copy link
Author

A-312 commented Mar 31, 2020

Why we need it?

You have right, there is no specific reason to use another plugin which does the same thing. This plugin should use the same of postcss-minify-params & postcss-unique-selectors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.