Homec4science

Perform commit message parsing and construction with new CustomFields

Authored by epriestley <git@epriestley.com> on Mar 8 2014, 02:05.

Description

Perform commit message parsing and construction with new CustomFields

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).

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T3886, T2222

Differential Revision: https://secure.phabricator.com/D8449

Details

Committed
epriestley <git@epriestley.com>Mar 11 2014, 21:01
Pushed
aubortJan 31 2017, 17:16
Parents
rPH966eb2ae26f6: Fail feed story renders individually, instead of in aggregate
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHae3c1f781939: Perform commit message parsing and construction with new CustomFields (authored by epriestley <git@epriestley.com>).Mar 11 2014, 21:01