Page MenuHomePhabricator

Specify branch in Diffusion commit email
AbandonedPublic

Authored by Krenair on May 17 2015, 10:42 PM.
Tags
None
Attached Files
Unknown Object (File)
Apr 6 2017, 1:27 PM
Unknown Object (File)
Mar 7 2017, 4:25 PM
Unknown Object (File)
Feb 9 2017, 6:21 AM
Unknown Object (File)
Jan 9 2017, 2:20 AM
Unknown Object (File)
Dec 27 2016, 5:10 AM
Unknown Object (File)
Dec 23 2016, 6:35 PM
Unknown Object (File)
Nov 27 2016, 6:15 AM
Unknown Object (File)
Nov 27 2016, 1:53 AM
Subscribers

Details

Summary

Fixed T6897

Test Plan

Commit to repository, check email inbox

Diff Detail

Repository
rP Phabricator
Branch
T6897
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 6052
Build 6072: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

Krenair retitled this revision from to Specify branch in Diffusion commit email.
Krenair updated this object.
Krenair edited the test plan for this revision. (Show Details)
Krenair added a reviewer: epriestley.

Did you actually execute your test plan? I would expect this change to:

  • not affect Diffusion mail at all; and
  • sometimes show a BRANCH heading twice in Differential mail.
epriestley edited edge metadata.
This revision now requires changes to proceed.May 19 2015, 2:14 PM

I checked the email quickly, but clearly not well enough.

I don't think I know enough about Git and Phabricator's internals to do this.