Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(730)

Issue 29768621: #10639 - node-sass output filter (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 2 weeks ago by l.rosilio
Modified:
2 weeks, 3 days ago
Reviewers:
mathias, Fred, f.lopez
Visibility:
Public.

Description

#10639 - node-sass output filter

Patch Set 1 #

Patch Set 2 : #10639 - node-sass output filter #

Patch Set 3 : #10639 - node-sass output filter #

Total comments: 1

Patch Set 4 : #10639 - node-sass output filter #

Total comments: 2

Patch Set 5 : #10639 - node-sass output filter #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
A edit.sh View 1 2 3 4 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 8
l.rosilio
2 months, 2 weeks ago (2018-05-02 12:48:41 UTC) #1
l.rosilio
2 months ago (2018-05-14 14:57:53 UTC) #2
l.rosilio
1 month, 1 week ago (2018-06-11 10:36:11 UTC) #3
mathias
https://codereview.adblockplus.org/29768621/diff/29804566/edit.sh File edit.sh (right): https://codereview.adblockplus.org/29768621/diff/29804566/edit.sh#newcode7 edit.sh:7: *Rendering*|*Wrote*) Can't you make these patterns more specific, in ...
1 month, 1 week ago (2018-06-11 14:04:14 UTC) #4
l.rosilio
1 month, 1 week ago (2018-06-12 14:56:04 UTC) #5
mathias
Almost there :) https://codereview.adblockplus.org/29768621/diff/29805567/edit.sh File edit.sh (right): https://codereview.adblockplus.org/29768621/diff/29805567/edit.sh#newcode7 edit.sh:7: *Rendering?Complete,?saving*file*|*Wrote?CSS?to?*) As discussed: Please remove the ...
1 month ago (2018-06-19 12:59:49 UTC) #6
l.rosilio
4 weeks, 1 day ago (2018-06-20 12:34:29 UTC) #7
mathias
4 weeks, 1 day ago (2018-06-20 12:53:14 UTC) #8
LGTM.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5