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
F15319832: D17805.id42818.diff
Thu, Mar 6, 9:29 PM
Unknown Object (File)
Thu, Feb 27, 8:30 AM
Unknown Object (File)
Sun, Feb 9, 3:56 AM
Unknown Object (File)
Sun, Feb 9, 3:56 AM
Unknown Object (File)
Sun, Feb 9, 3:56 AM
Unknown Object (File)
Sun, Feb 9, 3:56 AM
Unknown Object (File)
Feb 3 2025, 2:07 PM
Unknown Object (File)
Jan 27 2025, 2:27 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
Branch
comment1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16742
Build 22333: Run Core Tests
Build 22332: arc lint + arc unit