Skip to content

Conversation

@dutzi
Copy link

@dutzi dutzi commented Sep 6, 2019

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

We're using css-vars-ponyfill for IE11, we are a widget embedded in a website, we want css-vars-ponyfill to run only on CSS generated by us. We need a way to distinguish out CSS tags from others.

@jsf-clabot
Copy link

jsf-clabot commented Sep 6, 2019

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@codecov
Copy link

codecov bot commented Sep 6, 2019

Codecov Report

Merging #445 into master will increase coverage by 0.14%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #445      +/-   ##
==========================================
+ Coverage   88.32%   88.46%   +0.14%     
==========================================
  Files           4        4              
  Lines         411      416       +5     
  Branches       86       88       +2     
==========================================
+ Hits          363      368       +5     
  Misses         46       46              
  Partials        2        2              
Impacted Files Coverage Δ
src/index.js 88.28% <100.00%> (+0.26%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 159ce3b...5709b74. Read the comment docs.

@dutzi
Copy link
Author

dutzi commented Sep 6, 2019

Hey, I don't understand what Lint errors it's complaining about, can anyone help?
It only says:

Bash exited with code '1'.

Also, can't understand why tests are failing? Locally they run smoothly.

@dutzi
Copy link
Author

dutzi commented Sep 25, 2019

Any update?

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.

2 participants