Skip to content

Conversation

mgol
Copy link
Member

@mgol mgol commented Apr 28, 2025

3.x version of #577

Don't warn if jQuery.Deferred.getStackHook is set to the value of jQuery.Deferred.getErrorHook. This is to facilitate plugins supporting both jQuery <3.7 and older without triggering Migrate warnings and without requiring complex logic parsing jQuery.fn.jquery.

Ref gh-577

Don't warn if `jQuery.Deferred.getStackHook` is set to the value of
`jQuery.Deferred.getErrorHook`. This is to facilitate plugins supporting both
jQuery <3.7 and older without triggering Migrate warnings and without requiring
complex logic parsing `jQuery.fn.jquery`.

Ref jquerygh-577
@mgol mgol added this to the 3.6.0 milestone Apr 28, 2025
@mgol mgol self-assigned this Apr 28, 2025
mgol added a commit to mgol/jquery-migrate that referenced this pull request Apr 28, 2025
Don't warn if `jQuery.Deferred.getStackHook` is set to the value of
`jQuery.Deferred.getErrorHook`. This is to facilitate plugins supporting both
jQuery <3.7 and older without triggering Migrate warnings and without requiring
complex logic parsing `jQuery.fn.jquery`.

Ref jquerygh-578
@mgol mgol added the Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. label Apr 28, 2025
@timmywil timmywil removed Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. Needs review labels Apr 28, 2025
@mgol mgol merged commit 65978c9 into jquery:3.x-stable Apr 28, 2025
9 checks passed
@mgol mgol deleted the 3.x-getStackHook branch April 28, 2025 21:29
mgol added a commit that referenced this pull request Apr 28, 2025
Don't warn if `jQuery.Deferred.getStackHook` is set to the value of
`jQuery.Deferred.getErrorHook`. This is to facilitate plugins supporting both
jQuery <3.7 and older without triggering Migrate warnings and without requiring
complex logic parsing `jQuery.fn.jquery`.

Closes gh-577
Ref gh-578
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

Successfully merging this pull request may close these issues.

2 participants