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

fix dynamic insert-css usage #12

Merged
merged 3 commits into from
Aug 6, 2020
Merged

fix dynamic insert-css usage #12

merged 3 commits into from
Aug 6, 2020

Conversation

ahdinosaur
Copy link
Member

uses forked version of static-module: browserify/static-module#22

fixes choojs/bankai#109

uses forked version of `static-module`: browserify/static-module#22

fixes choojs/bankai#109
@codecov-io
Copy link

Current coverage is 97.36% (diff: 100%)

Merging #12 into master will not change coverage

@@             master        #12   diff @@
==========================================
  Files             1          1          
  Lines            38         38          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits             37         37          
  Misses            1          1          
  Partials          0          0          

Powered by Codecov. Last update f1e9254...f9518f6

@jondashkyle
Copy link

Just checking in on this one. Seems like static-module is going to be slow to merge any pull requests 😕

@yoshuawuyts
Copy link
Contributor

@jondashkyle I kinda feel that if this patch is needed too complex things are being done. Also no movement on the other patch, so think this is not going to land anytime soon (unless more work is done, but it's been a bit stale lately unforch)

Copy link
Member

@goto-bus-stop goto-bus-stop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed on main branch, gonna get this test in. thanks!

@goto-bus-stop goto-bus-stop merged commit f239045 into master Aug 6, 2020
@goto-bus-stop goto-bus-stop deleted the dynamic branch August 6, 2020 17:47
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.

breaks when insert-css usage isn't naively statically analyzable
5 participants