Bug 246035

Summary:	[webkitscmpy] Handle unexpected PR data
Product:	WebKit	Reporter:	Jonathan Bedard <jbedard>
Component:	Tools / Tests	Assignee:	Jonathan Bedard <jbedard>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	Other
Hardware:	Unspecified
OS:	Unspecified

Jonathan Bedard

Reported 2022-10-04 13:36:28 PDT

GitHub some times populates PR json data with unexpected 'None's in certain places. I'm not sure what the root cause is, I suspect it's related to creating a PR with a very large description. We should at least handle 'None' in more places in json data coming from GitHub.

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2022-10-04 13:41:56 PDT

<rdar://problem/100778857>

Jonathan Bedard

Comment 2 2022-10-04 13:43:44 PDT

Pull request: https://github.com/WebKit/WebKit/pull/4996

EWS

Comment 3 2022-10-14 13:42:15 PDT

Committed 255549@main (9dd26e36dd84): <https://commits.webkit.org/255549@main> Reviewed commits have been landed. Closing PR #4996 and removing active labels.

EWS

Comment 4 2022-10-18 13:14:57 PDT

Committed 252432.592@safari-7614-branch (d068f8be2383): <https://commits.webkit.org/252432.592@safari-7614-branch> Reviewed commits have been landed. Closing PR #143 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.