Homestyx hydra
Diffusion hydra 9574b91e55d4

Add options to include patches inline or attached for Diffusion commit emails

Description

Add options to include patches inline or attached for Diffusion commit emails

Summary:
See D818 for an older attempt at this. Support code has matured to the point where the patch is pretty straightforward.

@tido, this was a long-standing request from Aditya back in the day.

Test Plan: Used reparse.php --herald to send myself a bunch of emails with various patch configurations. Confirmed that limits are respected, reasonable errors arise when they're violated, etc. (Timeout is a little funky but that's out of scope here, I think.)

Reviewers: btrahan

Reviewed By: btrahan

CC: tido, aran

Maniphest Tasks: T456

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

Details

Provenance
epriestleyAuthored on Jul 12 2012, 4:33 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:2b690372dedc: Fix several issues with Differential exception email
Branches
Unknown
Tags
Unknown

Event Timeline