Page MenuHomePhabricator

underline rule - tweak it a bit
ClosedPublic

Authored by btrahan on Jan 17 2014, 9:46 PM.
Tags
None
Referenced Files
F14387531: D7999.id18098.diff
Sat, Dec 21, 2:50 PM
Unknown Object (File)
Thu, Dec 19, 5:04 PM
Unknown Object (File)
Wed, Dec 18, 6:23 AM
Unknown Object (File)
Sun, Dec 8, 6:58 AM
Unknown Object (File)
Tue, Nov 26, 7:34 AM
Unknown Object (File)
Nov 20 2024, 6:51 PM
Unknown Object (File)
Nov 19 2024, 7:41 AM
Unknown Object (File)
Nov 15 2024, 11:43 AM

Details

Summary

basically what you told me to do in D7995, but I also added a "/" to the 'don't capture' clause. In my chosen test code without this check we would end up underlining "and also", and I figured this special case will probably need to handle multiple files in a row with this formatting.

Of course, this means __/something__ also won't be underlined.

Test Plan

plugged the regex into a handy online tool (http://regex101.com/) and played with it a bunch. re-ran unit tests and they passed

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

The internal / seems maybe a tiny bit odd, but we can see what people hit and tweak as necessary.

Cool. I'm working on my orange belt. :D