Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29852577: Issue 6847 - Add regular expression support to snippets (Closed)

Created:
Aug. 11, 2018, 4:53 a.m. by Manish Jethani
Modified:
Aug. 16, 2018, 1:32 a.m.
Reviewers:
hub
CC:
kzar
Base URL:
https://hg.adblockplus.org/adblockpluscore/
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -4 lines) Patch
M lib/content/snippets.js View 3 chunks +46 lines, -4 lines 0 comments Download

Messages

Total messages: 5
Manish Jethani
Aug. 11, 2018, 4:53 a.m. (2018-08-11 04:53:59 UTC) #1
Manish Jethani
Patch Set 1
Aug. 11, 2018, 5:12 a.m. (2018-08-11 05:12:33 UTC) #2
hub
I wish we had test for that part. Otherwise, LGTM
Aug. 15, 2018, 9:28 p.m. (2018-08-15 21:28:01 UTC) #3
Manish Jethani
On 2018/08/15 21:28:01, hub wrote: > I wish we had test for that part. There ...
Aug. 15, 2018, 9:49 p.m. (2018-08-15 21:49:35 UTC) #4
hub
Aug. 16, 2018, 1:32 a.m. (2018-08-16 01:32:08 UTC) #5
Message was sent while issue was closed.
On 2018/08/15 21:49:35, Manish Jethani wrote:
> On 2018/08/15 21:28:01, hub wrote:
> > I wish we had test for that part.
> 
> There are no tests for the snippet library unfortunately. I think we'll have
to
> take this on as a separate task.

Yes that's unfortunate.

Powered by Google App Engine
This is Rietveld