Repository navigation
IE11 trailing comma issue in bundle.js #1881
Description
Getting an error regarding line 965 of bundle.js, because of a trailing comma.
I took a screen of this on my VM. Basically, the app works fine on most browsers, but on my IE11 VM, it freaks out because of this trailing comma. It's under the 'ansi-html/index.js' module - I'm not entirely sure where to look for this. I am running node-sass with CRA exactly how it's described in the readme, and also running react-scripts-ts.
Activity
Timer commentedon Mar 22, 2017
Please follow the issue template. It's there for a reason. 😉
You failed to mention
react-scriptsversion, among other things.I noticed you said this happens with
react-scripts-ts(but you used the world also), did you see if this replicates withreact-scripts?Did you open an issue with
react-scripts-tsfirst? We do not endorse nor provide support for other versions ofreact-scripts.gaearon commentedon Mar 22, 2017
Please also verify whether the problem exists in a newly created project with
react-scriptssincereact-scripts-tsis a fork and we don‘t know how up to date it is.trustfall commentedon Mar 22, 2017
Alright, sorry for missing the template guys. I've tried this with a fresh copy of CRA, with no scripts or any changes to the default CRA app, still producing the same issue.
react-scripts@0.9.5
npm - 3.10.10
node - 6.9.5
OS - IE11 VM from Microsoft
IE Version - 11.0.9600.18618
Just to add for specificity, this happens when running
yarn start, on the project, and navigating to10.0.2.2:3000in my VM, which points to localhost (default behavior/address in VM).gaearon commentedon Mar 22, 2017
Tagging up for grabs. Need to find why the package has a trailing comma and maybe file an issue with the package or use an alternative.
Timer commentedon Mar 23, 2017
The trailing comma was removed in v0.0.6 and newer.
Timer commentedon Mar 26, 2017
@syn3r6y were you interested in submitting a PR updating this package? 😀
trustfall commentedon Mar 27, 2017
Yeah, I can probably take a look today or tomorrow
Timer commentedon Mar 31, 2017
Let me know if you need help with anything. 😄
Update ansi-html to fix #1881
Timer commentedon Apr 15, 2017
Fixed in 4b92fd3.
Merge upstream changes from create-react-app 0.9.x (#84)
Update ansi-html to fix react#1881