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

JavaScript error due to missing semicolon #70

Closed
mahboob-awan opened this issue Dec 22, 2014 · 3 comments
Closed

JavaScript error due to missing semicolon #70

mahboob-awan opened this issue Dec 22, 2014 · 3 comments

Comments

@mahboob-awan
Copy link

Please add semicolon at the end of minified file because missing semicolon caused javascript error at following place:

!function(e,t){ /* bowser minified code*/}) 

/*!
 * jQuery Migrate - v1.2.1 - 2013-05-08
 * https://github.com/jquery/jquery-migrate
 * Copyright 2005, 2013 jQuery Foundation, Inc. and other contributors; Licensed MIT
 */
(function( jQuery, window, undefined ) {...})( jQuery, window );

@jasonkarns
Copy link

the semicolon is not required and would not cause a js error on its own. it could cause an error if concatenating with other scripts. if that is the case, I would recommend using a tool that concatenates intelligently by inserting semicolons between concatenated scripts.

@ded
Copy link
Collaborator

ded commented Aug 8, 2015

same.

@ded ded closed this as completed Aug 8, 2015
@uaoleg
Copy link

uaoleg commented Feb 3, 2017

Is it a problem to add a semicolon and to reduce the amount of butthurt of lots of people?

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

4 participants