🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@mgol
Copy link
Member

@mgol mgol commented Jan 4, 2022

Summary

This is a version of gh-4993 for the 3.x-stable branch.

The GitHub UI treats #NUMBER as referring to its own issues which is confusing
when in jQuery source it's usually referring to the old deprecated Trac instance
at https://bugs.jquery.com. This change replaces all such Trac references with
trac-NUMBER.

A few of the references came with the Sizzle integration and referred to the
Sizzle GitHub bug tracker. Those have been replaced with full links instead.

A new entry describing issue reference conventions has been added to README.

Ref gh-4993
Ref 5d5ea01

Note: this PR already includes changes from #4995.

Checklist

  • New tests have been added to show the fix or feature works
  • Grunt build and unit tests pass locally with these changes
  • If needed, a docs issue/PR was created at https://github.com/jquery/api.jquery.com

@mgol mgol added the Docs label Jan 4, 2022
@mgol mgol added this to the 3.6.1 milestone Jan 4, 2022
@mgol mgol requested a review from timmywil January 4, 2022 16:03
@mgol mgol self-assigned this Jan 4, 2022
@mgol mgol added Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. Needs review labels Jan 4, 2022
This is a version of jquerygh-4993 for the `3.x-stable` branch.

The GitHub UI treats `#NUMBER` as referring to its own issues which is confusing
when in jQuery source it's usually referring to the old deprecated Trac instance
at https://bugs.jquery.com. This change replaces all such Trac references with
`trac-NUMBER`.

A few of the references came with the Sizzle integration and referred to the
Sizzle GitHub bug tracker. Those have been replaced with full links instead.

A new entry describing issue reference conventions has been added to README.

Ref jquerygh-4993
Ref 5d5ea01
@timmywil timmywil removed the Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. label Jan 6, 2022
Copy link
Member

@timmywil timmywil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, as long as you did the same thing with issues numbers less than 5000

@mgol
Copy link
Member Author

mgol commented Jan 7, 2022

Yes, I backported all the changes from #4995 to here as well.

@mgol mgol merged commit 95e34b6 into jquery:3.x-stable Jan 12, 2022
@mgol mgol deleted the 3.x-trac-links branch January 12, 2022 22:23
@mgol mgol removed the Needs review label Jan 12, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

2 participants