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
F13818715: D17805.diff
Fri, Sep 20, 9:41 AM
Unknown Object (File)
Sun, Sep 1, 5:19 AM
Unknown Object (File)
Aug 19 2024, 3:29 AM
Unknown Object (File)
Aug 15 2024, 2:33 PM
Unknown Object (File)
Aug 11 2024, 2:59 AM
Unknown Object (File)
Aug 1 2024, 6:39 AM
Unknown Object (File)
Jul 26 2024, 2:00 PM
Unknown Object (File)
Jul 24 2024, 9:19 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