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

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

Tags

None

Referenced Files

	F13276519: D8514.diff
	Fri, May 31, 6:06 AM

	F13267008: D8514.id20195.diff
	Tue, May 28, 11:34 PM

	F13264977: D8514.id.diff
	Tue, May 28, 12:10 AM

	F13264976: D8514.id20196.diff
	Tue, May 28, 12:10 AM

	F13263053: D8514.id.diff
	Mon, May 27, 6:47 AM

	F13227853: D8514.diff
	Mon, May 20, 5:17 AM

	F13207218: D8514.diff
	Wed, May 15, 10:00 PM

	F13190908: D8514.id20196.diff
	Sat, May 11, 3:15 PM

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