Page MenuHomePhabricator

Perform commit message parsing and construction with new CustomFields
ClosedPublic

Authored by epriestley on Mar 8 2014, 5:01 PM.
Tags
None
Referenced Files
F13968460: D8449.diff
Wed, Oct 16, 9:12 PM
Unknown Object (File)
Sun, Oct 13, 10:40 PM
Unknown Object (File)
Sep 12 2024, 9:16 AM
Unknown Object (File)
Sep 9 2024, 3:15 PM
Unknown Object (File)
Sep 6 2024, 10:31 AM
Unknown Object (File)
Aug 29 2024, 5:45 PM
Unknown Object (File)
Aug 26 2024, 2:59 AM
Unknown Object (File)
Aug 19 2024, 11:12 PM
Subscribers

Details

Summary

Ref T2222. Ref T3886. Converts parsing and construction of commit messages to be driven by CustomField.

Test Plan

This is a huge, messy change. I've made an effort to test it exhasutively, but suspect I probably missed a few behaviors. Roughly:

  • Enumerted all current fields (fields implementing shouldAppearOnCommitMessage()) and tried to test them one by one.
  • Used arc diff --edit repeatedly to manipulate each field (this workflow hits both the parse and construct steps).
  • Used arc amend --show to examine construct output (this does not activate the "edit" mode).

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 10 2014, 4:24 PM
epriestley updated this revision to Diff 20114.

Closed by commit rPae3c1f781939 (authored by @epriestley).