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

Latest version causes test failures when used in electron-compile(rs) #4

Closed
2 tasks
maxkorp opened this issue Oct 23, 2015 · 4 comments
Closed
2 tasks

Comments

@maxkorp
Copy link
Contributor

maxkorp commented Oct 23, 2015

So i forked all 3 projects, and updated this with the contents of #3.
I then pointed electron-compilers at the latest version, and then pointed electron-compile at the latest version of this, as well as the version of compilers i just mentioned.

2 Tests fail:

  • 'The inline HTML compiler' - 'should remove protocol-relative URLs because they are dumb'
  • 'The coffeescript compiler' - 'should fail on invalid.coffee'

Let me know if you'd rather have this issue logged in either of those modules instead of/ along with here

@anaisbetts
Copy link
Contributor

@maxkorp Nope, this is fine, I'm going to work on this today as well as a few other improvements to compile-cache and friends

@maxkorp
Copy link
Contributor Author

maxkorp commented Oct 23, 2015

Cool, I've got a few small PRs to make as well (cleaning lib up, etc). Thanks for throwing all this together. It's been a huge boon for me and my team.

@maxkorp
Copy link
Contributor Author

maxkorp commented Oct 23, 2015

Worth noting, if I manually go in and make the changes from #3 to the installed copies of 0.6.3 (in both compile and compilers), tests still work just fine, so it's not that change. Something else between 0.6.3 and master.

@anaisbetts
Copy link
Contributor

Both of these failures are fixed (and were caused by isMinified incorrectly deciding that small files are minified)

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

No branches or pull requests

2 participants