Skip to content

Fix #13274: wrap sourceMap directive in multiline comments #1143

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Fix #13274: wrap sourceMap directive in multiline comments #1143

wants to merge 2 commits into from

Conversation

gibson042
Copy link
Member

Sizes - compared to 1.9-stable @ 21af3a9

    267328         (-)  dist/jquery.js                                         
    138252         (-)  dist/jquery.min.map                                    
     92988         (-)  dist/jquery.min.js                                     
     32963        (-1)  dist/jquery.min.js.gz

@dmethvin
Copy link
Member

Sweet! You need to add cc_on.html to the PR.

@gibson042
Copy link
Member Author

Details, details...

@jaubourg
Copy link
Member

@gibson042 old people are obsessed with details!

/me hides from @dmethvin's wrath

@dmethvin
Copy link
Member

What in tarnation?! You boys get off my lawn!

@gibson042
Copy link
Member Author

After you give us back our ball, mister!

@jaubourg
Copy link
Member

Look, sir, we didn't drop the ball... @danheberden did... as usual... :/

gibson042 added a commit that referenced this pull request Jan 25, 2013
@gibson042
Fix #13274: Wrap sourceMap directive in multiline comments. Close gh-…
ac93559 
…1143.
@gibson042 gibson042 closed this in 487b703 Jan 25, 2013
mescoda pushed a commit to mescoda/jquery that referenced this pull request Nov 4, 2014
@gibson042
Fix #13274: Wrap sourceMap directive in multiline comments. Close jqu…
cea165c 
…erygh-1143.
@lock lock bot locked as resolved and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gibson042 @dmethvin @jaubourg