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

Inventory update - unexpected token ILLEGAL #1434

Closed
g8d3 opened this issue Aug 1, 2015 · 3 comments
Closed

Inventory update - unexpected token ILLEGAL #1434

g8d3 opened this issue Aug 1, 2015 · 3 comments

Comments

@g8d3
Copy link
Contributor

g8d3 commented Aug 1, 2015

Challenge http://www.freecodecamp.com/challenges/bonfire-inventory-update has an issue.

I am getting unexpected token ILLEGAL in this challenge with this code.

When I run that on node console I don't get any error.

It's working here too.

@nree
Copy link

nree commented Aug 1, 2015

https://gist.github.com/juanpastas/5e694465eb624f0b9fde#file-inv-js-L19
https://gist.github.com/juanpastas/5e694465eb624f0b9fde#file-inv-js-L21

those two lines are being mangled because it is being picked up as comments, it's the result of quick code clean up for a hopefully easier time to run the JS code as far as the tests go

the issue exists here:
https://github.com/FreeCodeCamp/freecodecamp/blob/staging/public/js/lib/coursewares/coursewaresJSFramework_0.0.6.js#L156

@g8d3
Copy link
Contributor Author

g8d3 commented Aug 1, 2015

I also removed c(x) at the end, thanks @secretofmana.

@g8d3 g8d3 closed this as completed Aug 1, 2015
@g8d3 g8d3 reopened this Aug 1, 2015
@g8d3
Copy link
Contributor Author

g8d3 commented Aug 1, 2015

Here are some tests failing https://jsbin.com/fayezajihi/edit?js,console.

And passing https://jsbin.com/racoyi/edit?js,console

@secretofmana thanks. I will create PR.

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

No branches or pull requests

2 participants