Bug 230655 - [webkitscmpy] Linkify http urls in pull-requests
Summary: [webkitscmpy] Linkify http urls in pull-requests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-09-22 17:02 PDT by Jonathan Bedard
Modified: 2021-10-01 09:27 PDT (History)
2 users (show)

See Also:


Attachments
Patch (14.23 KB, patch)
2021-09-29 16:52 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch for landing (15.58 KB, patch)
2021-10-01 08:55 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2021-09-22 17:02:11 PDT
In GitHub pull requests, we should linkify http urls. This includes both the code to generate the pull-request and the code parsing our existing pull-requests.
Comment 1 Jonathan Bedard 2021-09-22 17:16:36 PDT
Pull-request: https://github.com/WebKit/WebKit/pull/10
Comment 2 Jonathan Bedard 2021-09-29 16:52:55 PDT
Created attachment 439681 [details]
Patch
Comment 3 Radar WebKit Bug Importer 2021-09-29 17:03:37 PDT
<rdar://problem/83695409>
Comment 4 Stephanie Lewis 2021-09-30 17:40:44 PDT
Comment on attachment 439681 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=439681&action=review

> Tools/Scripts/libraries/webkitscmpy/webkitscmpy/pull_request.py:52
> +    def escape_html(cls, message):

out of curiosity why not use an escape library?
Comment 5 Jonathan Bedard 2021-10-01 08:22:32 PDT
Comment on attachment 439681 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=439681&action=review

>> Tools/Scripts/libraries/webkitscmpy/webkitscmpy/pull_request.py:52
>> +    def escape_html(cls, message):
> 
> out of curiosity why not use an escape library?

I didn't find one that did exactly what we needed, because we don't need general HTML escaping (GitHub does quite a bit of the work for us), just some very specific cases to handle string like "<rdar://problem/1234">
Comment 6 Jonathan Bedard 2021-10-01 08:55:08 PDT
Created attachment 439865 [details]
Patch for landing
Comment 7 EWS 2021-10-01 09:27:31 PDT
Committed r283373 (242381@main): <https://commits.webkit.org/242381@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 439865 [details].