Silence a warning about unknown fields in 'getcommitmessage'
ClosedPublic
Actions

Authored by epriestley on Mar 13 2014, 12:05 AM.

Tags

None

Referenced Files

	F18110111: D8514.diff
	Mon, Aug 11, 4:19 PM

	F18107094: D8514.id.diff
	Sun, Aug 10, 11:39 PM

	F18097843: D8514.diff
	Fri, Aug 8, 3:50 PM

	F18031284: D8514.id20196.diff
	Sat, Aug 2, 11:57 PM

	F17946457: D8514.id.diff
	Thu, Jul 31, 2:33 PM

	F17944857: D8514.id20195.diff
	Thu, Jul 31, 11:00 AM

	F17923611: D8514.diff
	Wed, Jul 30, 8:27 AM

	Unknown Object (File)
	Jun 20 2025, 11:21 AM

Subscribers

Details

Reviewers

zeeg
btrahan
dctrwatson

Commits

Restricted Diffusion Commit
rPe068b04b5c95: Silence a warning about unknown fields in 'getcommitmessage'

Summary

Similar to D8491, some of my new exceptions are a bit too aggressive. See IRC. This one's hitting an edit workflow with 'revisionID' onboard somehow.

Test Plan

Not entirely sure how to hit this, but it won't throw anymore.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20195.Mar 13 2014, 12:05 AM

epriestley retitled this revision from to Silence a warning about unknown fields in 'getcommitmessage'.

epriestley updated this object.

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

epriestley added reviewers: btrahan, dctrwatson.

Herald added subscribers: epriestley, aran.Mar 13 2014, 12:05 AM

can i haz accept? ;)

This revision is now accepted and ready to land.Mar 13 2014, 12:10 AM

Closed by commit rPe068b04b5c95 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

conduit/

ConduitAPI_differential_getcommitmessage_Method.php

8 lines

Diff 20196

View Options

src/applications/differential/conduit/ConduitAPI_differential_getcommitmessage_Method.php

Silence a warning about unknown fields in 'getcommitmessage'ClosedPublicActions