Homec4science

Move Differential commit message parsing to a separate, tested class

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

Description

Move Differential commit message parsing to a separate, tested class

Summary: Ref T2222. We have a hunk of logic that purely does text parsing here; separate it and get coverage on it.

Test Plan:

  • Ran new unit tests.
  • Used differential.parsecommitmessage.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2222

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

Details

Committed
epriestley <git@epriestley.com>Mar 8 2014, 02:44
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf25cce1e695b: Remove DifferentialCommentEditor and DifferentialCommentMail
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHaff34077c52c: Move Differential commit message parsing to a separate, tested class (authored by epriestley <git@epriestley.com>).Mar 8 2014, 02:44