FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Failing test + fix for backtick quotes by TehShrike · Pull Request #1785 · browserify/browserify · GitHub

Repository navigation

Failing test + fix for backtick quotes - #1785

Merged
goto-bus-stop merged 2 commits into
browserify:masterfrom
TehShrike:backticks-test
Jan 4, 2018
Merged

goto-bus-stop merged 2 commits into
browserify:masterfrom
TehShrike:backticks-test

Conversation

TehShrike commented Dec 20, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor

Issue #1782

This issue needs to be fixed in module-deps. I will open a pull request there.

Copy link
Copy Markdown
Member

in https://github.com/browserify/detective even, i think. thanks for working on this :D

Copy link
Copy Markdown
Contributor Author

I'll update this to use module-deps@2.x and then it should be good.

TehShrike changed the title Failing test for backtick quotes Failing test + fix for backtick quotes Jan 2, 2018

Copy link
Copy Markdown
Contributor Author

Bumping the dependency on module-deps guarantees that the backtick-supporting version of detective will be used.

The tests fail in 0.10 and 0.12 because detective doesn't support those versions any more.

Copy link
Copy Markdown
Member

merging since the failures are only on node <4 and tests will pass on master. thanks!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL