Remove unnecessary internal use of jQuery.trim #3095
Conversation
|
By analyzing the blame information on this pull request, we identified @farmdawgnation, @jaubourg and @gibson042 to be potential reviewers |
|
Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA). 📝 Please visit http://contribute.jquery.org/CLA/ to sign. After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know. If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check. |
ralinc
force-pushed
the
remove-whitespace-trim
branch
from
07cb37c to
bb374cd
Compare
As agreed in the issue, the call to jQuery.trim can be removed since the subsequent match will ignore leading/trailing space anyway. Fixes jquery#3003.
ralinc
force-pushed
the
remove-whitespace-trim
branch
from
bb374cd to
833f444
Compare
|
@dmethvin I can remove the "Fixes 3003" from the description if you want to merge this PR. |
|
Ticket reference updated and landed; thanks @ralinchimev! |
5 participants
Summary
The call to jQuery.trim can be removed since the subsequent match
will ignore leading and trailing spaces anyway. Fixes #3003.
Checklist
Mark an
[x]for completed items, if you're not sure leave them unchecked and we can assist.