| Summary: | Bugzilla needs to linkify identifiers | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||||
| Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||||
| Status: | RESOLVED FIXED | ||||||||||
| Severity: | Normal | CC: | aakash_jain, ap, dewei_zhu, lingho, webkit-bug-importer | ||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||
| Version: | WebKit Nightly Build | ||||||||||
| Hardware: | Unspecified | ||||||||||
| OS: | Unspecified | ||||||||||
| Attachments: |
|
||||||||||
|
Description
Jonathan Bedard
2021-04-07 16:43:45 PDT
Created attachment 425455 [details]
Patch
(In reply to Jonathan Bedard from comment #2) > Created attachment 425455 [details] > Patch I'm intend to include this change in the same extension used to linkify revisions. First step is to rename the extension. *** Bug 224325 has been marked as a duplicate of this bug. *** Created attachment 426964 [details]
Patch
Comment on attachment 426964 [details]
Patch
r=me
Committed r276536 (236983@main): <https://commits.webkit.org/236983@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 426964 [details]. Reopening to attach new patch. Created attachment 427062 [details]
Patch
Comment on attachment 427062 [details]
Patch
rs=me
Is the linking to commits.webkit.org long-term solution, or after we move to github, we should link directly to github instead of using one translation layer in between?
(In reply to Aakash Jain from comment #10) > Comment on attachment 427062 [details] > Patch > > rs=me > > Is the linking to commits.webkit.org long-term solution, or after we move to > github, we should link directly to github instead of using one translation > layer in between? That's our long-term solution, because we don't expect that GitHub will ever support identifiers natively. Committed r276640 (237068@main): <https://commits.webkit.org/237068@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 427062 [details]. Deployed this to bugs.webkit.org |