grunt-ts icon indicating copy to clipboard operation
grunt-ts copied to clipboard

Add "### Fast Compile >> .....ts" behind verbose flag, or add silent flag?

Open ThaNarie opened this issue 10 years ago • 6 comments
trafficstars

Hi,

One of the usages of grunt-ts is to compile 'generated' typescript files by a task that rune before this so all code is valid js.

For example:

scaffold:
  - 'scaffold-pages'
  - 'ts'
  - 'sass'
  - 'autoprefixer'
...
### Fast Compile >>deploy/htdocs/inc/script/test/jasmine-matchers.d.ts 
### Fast Compile >>deploy/htdocs/inc/script/test/jasmine.d.ts          
### Fast Compile >>deploy/htdocs/inc/script/test/testSpec.ts           
...

But because the output of grunt-ts is so 'long', as a user you are missing the output of the 'main' task (scaffold-pages).

It would be awesome if we can configure the verbosity of the task, either by putting this output behind the verbose option, or maybe adding a silent option to keep this the default but providing an option to keep it quiet :)

I'm open for providing a PR if desired, but just wanted to know if you're up to this idea and which of the above would be the best approach.

ThaNarie avatar Dec 09 '14 21:12 ThaNarie

It would be awesome if we can configure the verbosity of the task,

Do-able without an option > If the count of the filtered files for fast compile equate the original file count > means we are not actively fast compiling and don't output these file names. PR much appreciated.

basarat avatar Dec 10 '14 05:12 basarat

Looking at the docs again, I guess I should just set fast to "never" as it will never fast compile when doing a 'stand-alone' grunt-ts from the command-line, and just re-generates the cache every time.

But a change in output when people like me are leaving the option to the default value could still be a good addition, so I might look into that :)

ThaNarie avatar Dec 10 '14 09:12 ThaNarie

But a change in output when people like me are leaving the option to the default value could still be a good addition, so I might look into that

:+1: leaving this issue open for a hero

basarat avatar Dec 10 '14 23:12 basarat

+1 I would also like to have that, however, until end of this year I might be too busy to look into that.

0815fox avatar Apr 15 '16 12:04 0815fox

Dear god do want. +1.

I just converted a problem from old-style /// <reference path="..." />s to imports and the wall of blue is very disorienting.

JoshuaKGoldberg avatar May 08 '16 20:05 JoshuaKGoldberg

I do intend to fix this - in the meantime it should be fairly straight forward to comment out the line that outputs it - it should be around line 39 in compile.js, depending on which version you have.

nycdotnet avatar May 10 '16 12:05 nycdotnet