Page MenuHomePhabricator
Differential D10270

Remarkup - add a regex to blacklist what objects get link
ClosedPublic
Actions

Authored by btrahan on Aug 14 2014, 10:08 PM.
Tags
None
Referenced Files
F19072727: D10270.id.diff
Sun, Nov 30, 11:30 PM
F19058786: D10270.diff
Sat, Nov 29, 3:23 AM
F18832796: D10270.id.diff
Oct 25 2025, 11:07 PM
F18826343: D10270.diff
Oct 24 2025, 3:31 AM
F18766250: D10270.id24730.diff
Oct 7 2025, 3:45 PM
F18712095: D10270.id.diff
Sep 29 2025, 5:16 AM
F18711409: D10270.diff
Sep 29 2025, 4:05 AM
F18589483: D10270.diff
Sep 12 2025, 3:27 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T5453: Blacklist monograms which are also used in normal discussions, like "Q1" (quarter 1)
Commits
Restricted Diffusion Commit
rPff51a1a451f1: Remarkup - add a regex to blacklist what objects get link
Summary

Fixes T5453.

Test Plan

made a remarkup comment that "Q1 is dumb and Q10 is awesome" and only Q10 was linked. changed the new setting to have the value " " and the Q1 also started linking.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan updated this revision to Diff 24730.Aug 14 2014, 10:08 PM
btrahan added a task: T5453: Blacklist monograms which are also used in normal discussions, like "Q1" (quarter 1).
btrahan retitled this revision from to Remarkup - add a regex to blacklist what objects get link.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 14 2014, 10:08 PM
Harbormaster completed remote builds in B2233: Diff 24730.Aug 14 2014, 10:09 PM
epriestley accepted this revision.Aug 14 2014, 10:11 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 14 2014, 10:11 PM
btrahan closed this revision.Aug 14 2014, 10:17 PM
btrahan updated this revision to Diff 24731.

Closed by commit rPff51a1a451f1 (authored by @btrahan).

Revision Contents
Changeset List

PathSize
src/
applications/
config/
option/
12 lines
ponder/
remarkup/
12 lines
docs/
user/
userguide/
10 lines
infrastructure/
markup/
rule/
6 lines

Diff 24731

View Options

src/applications/config/option/PhabricatorCoreConfigOptions.php

Loading...
View Options

src/applications/ponder/remarkup/PonderRemarkupRule.php

Loading...
View Options

src/docs/user/userguide/remarkup.diviner

Loading...
View Options

src/infrastructure/markup/rule/PhabricatorObjectRemarkupRule.php

Loading...
Phabricator · Built by Phacility