Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

Added: postcss-image-set-polyfill (#43) #373

Merged
merged 2 commits into from
May 7, 2017
Merged

Added: postcss-image-set-polyfill (#43) #373

merged 2 commits into from
May 7, 2017

Conversation

SuperOl3g
Copy link
Contributor

Solves #43

@SuperOl3g
Copy link
Contributor Author

SuperOl3g commented May 1, 2017

Aaauch... not working with ES6, will fix it tomorrow

But some tests isn't also passed in original repo

not ok 95 should add autoprefixer support
  ---
    operator: equal
    expected: |-
      '* {\n  -webkit-transition: -webkit-transform 1s;\n  transition: -webkit-transform 1s;\n  transition: transform 1s;\n  transition: transform 1s, -webkit-transform 1s;\n}\n\n@-webkit-keyframes spin {\n  0% { -webkit-transform: rotate(0deg); transform: rotate(0deg) }\n  100% { -webkit-transform: rotate(360deg); transform: rotate(360deg) }\n}\n\n@keyframes spin {\n  0% { -webkit-transform: rotate(0deg); transform: rotate(0deg) }\n  100% { -webkit-transform: rotate(360deg); transform: rotate(360deg) }\n}'
    actual: |-
      '* {\n  transition: -webkit-transform 1s;\n  transition: transform 1s;\n  transition: transform 1s, -webkit-transform 1s;\n}\n\n@-webkit-keyframes spin {\n  0% { -webkit-transform: rotate(0deg); transform: rotate(0deg) }\n  100% { -webkit-transform: rotate(360deg); transform: rotate(360deg) }\n}\n\n@keyframes spin {\n  0% { -webkit-transform: rotate(0deg); transform: rotate(0deg) }\n  100% { -webkit-transform: rotate(360deg); transform: rotate(360deg) }\n}'
    at: testFeature (/home/travis/build/MoOx/postcss-cssnext/lib/__tests__/option.features.js:41:5)

@MoOx
Copy link
Owner

MoOx commented May 6, 2017

Some tests are probably out of date. Feel free to make a PR to fix those :)

@Semigradsky
Copy link
Collaborator

This error happens because browserslist defaults is dynamic. We should pass browsers option in tests.

@Semigradsky Semigradsky added this to the v2.11.0 milestone May 7, 2017
@Semigradsky
Copy link
Collaborator

I have fixed it in 2ca6df0

@Semigradsky Semigradsky merged commit 08dd185 into MoOx:master May 7, 2017
@Semigradsky
Copy link
Collaborator

@MoOx
Copy link
Owner

MoOx commented May 9, 2017

Yeah why not. Do you want me to cut a release from master? Not sure why you created a milestone with an issue that's look fixed in master.

@Semigradsky
Copy link
Collaborator

I just forgot to close issue.

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.

3 participants