HomePhorge

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, 1:33 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP2b690372dedc: Fix several issues with Differential exception email
Branches
Unknown
Tags
Unknown

Event Timeline