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

Modify the error Regular expressions js in querystringvalues.js #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

foru17
Copy link

@foru17 foru17 commented Oct 3, 2015

There is a error regular expressions js in the querystringvalues.js which result in a Uncaught SyntaxError.

@bgrins
Copy link
Owner

bgrins commented Oct 3, 2015

Hi, thanks for catching this. It looks like the js file itself is fine (https://github.com/bgrins/devtools-snippets/blob/master/snippets/querystringvalues/querystringvalues.js) so this is likely an error in the build step that grabs the file and renders it to HTML (somewhere around here): https://github.com/bgrins/devtools-snippets/blob/master/Rakefile#L21. A fix in the build would be ideal so that this doesn't get wiped out next time rake build is run

@gangsthub
Copy link

Hi, @bgrins!

It's still wrong ni the output, though.

Cheers, and thanks for this repo ;)

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.

None yet

3 participants