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
Unknown Object (File)
Sun, Dec 15, 4:21 AM
Unknown Object (File)
Wed, Dec 4, 3:52 PM
Unknown Object (File)
Tue, Dec 3, 11:08 PM
Unknown Object (File)
Thu, Nov 28, 8:57 AM
Unknown Object (File)
Wed, Nov 27, 10:21 AM
Unknown Object (File)
Tue, Nov 19, 7:38 PM
Unknown Object (File)
Nov 7 2024, 11:56 AM
Unknown Object (File)
Nov 7 2024, 11:03 AM
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).