Page MenuHomePhabricator

Only recognize "Fixes ..." in main revision content like the Summary or Test Plan
ClosedPublic

Authored by epriestley on Apr 30 2017, 7:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 27, 2:27 AM
Unknown Object (File)
Fri, Jan 24, 8:34 PM
Unknown Object (File)
Fri, Jan 24, 8:34 PM
Unknown Object (File)
Fri, Jan 24, 8:34 PM
Unknown Object (File)
Fri, Jan 24, 8:34 PM
Unknown Object (File)
Fri, Jan 24, 12:49 PM
Unknown Object (File)
Dec 31 2024, 5:41 PM
Unknown Object (File)
Dec 30 2024, 6:37 AM
Subscribers
None

Details

Summary

Fixes T12642. Currently, writing "Fixes T..." in a comment gets picked up as a formal "fixes".

This is a bit confusing, and can also give you a "no effect" error if you "fixes ..." a task which is already "fixes"'d.

We could make the duplicate action a non-error, but just prevent the text from having an effect instead, which seems cleaner.

Test Plan
  • Wrote "Fixes ..." in a summary, saw a "fixes" relationship established.
  • Wrote "Fixes ..." in a comment, got a "mention" instead.
  • var_dump()'d some stuff as a sanity check, looked reasonable.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable