Page MenuHomePhabricator

Fix an issue where Herald may fail to extract content from an empty commit
ClosedPublic

Authored by epriestley on Apr 20 2022, 4:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 24, 12:30 AM
Unknown Object (File)
Feb 15 2024, 12:50 PM
Unknown Object (File)
Feb 11 2024, 2:42 AM
Unknown Object (File)
Feb 4 2024, 1:41 AM
Unknown Object (File)
Feb 4 2024, 1:37 AM
Unknown Object (File)
Jan 27 2024, 8:37 AM
Unknown Object (File)
Dec 22 2023, 6:41 PM
Unknown Object (File)
Dec 8 2023, 7:11 PM
Subscribers
None

Details

Summary

Ref T13667. The Herald "content added" rule (and other similar rules) do not correctly extract content from empty commits.

When we load an empty raw diff, return an empty changed content map.

Ref T13588. Also fix some PHP8.1 null/string stuff

Test Plan
  • Ran "bin/repository reparse --publish <commit>", with an empty commit hash and a nonempty commit hash.
  • Reviewed Herald transcripts for general sanity.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable