Paths

Table of Contentst

Actually squelch Harbormaster "test passed" mail
ClosedPublic
Actions

Authored by epriestley on Apr 18 2014, 11:31 PM.

Tags

None

Referenced Files

	F15447377: D8813.id20913.diff
	Thu, Mar 27, 11:26 PM

	F15445132: D8813.id20917.diff
	Thu, Mar 27, 12:30 PM

	F15444695: D8813.id.diff
	Thu, Mar 27, 10:45 AM

	F15442891: D8813.diff
	Thu, Mar 27, 1:08 AM

	F15397240: D8813.id20913.diff
	Sun, Mar 16, 7:06 PM

	F15393762: D8813.diff
	Sat, Mar 15, 11:06 PM

	F15388681: D8813.id20913.diff
	Sat, Mar 15, 4:15 AM

	F15388390: D8813.id20917.diff
	Sat, Mar 15, 3:46 AM

View All Files

Subscribers

epriestley

Tokens

"Baby Tequila" token, awarded by btrahan.

Details

Reviewers

btrahan

Maniphest Tasks

T1049: Implement Harbormaster

Commits

Restricted Diffusion Commit
rP9889892e5b76: Actually squelch Harbormaster "test passed" mail

Summary

Ref T1049. When Harbormaster tests pass, don't bother sending an email about it.

(I tried to implement this earlier but didn't test it entirely properly, and we needed a little more code.)

Test Plan

Used bin/harbormaster build to build some junk, got no email about passes.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20913.Apr 18 2014, 11:31 PM

epriestley retitled this revision from to Actually squelch Harbormaster "test passed" mail.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T1049: Implement Harbormaster.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 18 2014, 11:31 PM

Harbormaster completed remote builds in B8: Diff 20913.Apr 18 2014, 11:33 PM

btrahan accepted this revision.Apr 19 2014, 12:24 AM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Apr 19 2014, 12:24 AM

Closed by commit rP9889892e5b76 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

storage/

DifferentialTransaction.php

2 lines

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

15 lines

Diff 20917

View Options

src/applications/differential/storage/DifferentialTransaction.php