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
F19071220: D8514.diff
Sun, Nov 30, 6:58 PM
F18849807: D8514.id20196.diff
Oct 30 2025, 12:22 PM
F18845879: D8514.id.diff
Oct 29 2025, 2:52 PM
F18704164: D8514.id20196.diff
Sep 28 2025, 6:26 AM
F18645270: D8514.diff
Sep 19 2025, 7:17 AM
F18621720: D8514.diff
Sep 15 2025, 9:15 AM
F18609660: D8514.diff
Sep 14 2025, 3:41 AM
F18594301: D8514.diff
Sep 12 2025, 5:45 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).