Page MenuHomePhabricator

Silence a warning about unknown fields in 'getcommitmessage'
ClosedPublic

Authored by epriestley on Mar 13 2014, 12:05 AM.
Tags
None
Referenced Files
F19819538: D8514.id.diff
Mar 6 2026, 7:13 AM
F19720629: D8514.diff
Feb 12 2026, 10:06 PM
F19582566: D8514.id20196.diff
Feb 1 2026, 10:48 PM
F19539091: D8514.id20196.diff
Jan 21 2026, 5:18 PM
F19530596: D8514.id20195.diff
Jan 19 2026, 11:16 PM
F19527154: D8514.id20195.diff
Jan 18 2026, 11:19 PM
F19071395: D8514.id20195.diff
Nov 30 2025, 7:28 PM
F19071394: D8514.id20196.diff
Nov 30 2025, 7:28 PM

Details

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 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.
zeeg added a reviewer: zeeg.
zeeg added a subscriber: zeeg.

can i haz accept? ;)

This revision is now accepted and ready to land.Mar 13 2014, 12:10 AM
epriestley updated this revision to Diff 20196.

Closed by commit rPe068b04b5c95 (authored by @epriestley).