Make Nimib logging to stdout optional (-d:nimibQuiet)

[neroist]

I'd rather not have Nimib print to stdout all the time & fill up the terminal (esp. when compiling multiple documents!), and its better for users to have the option to disable logging.

[HugoGranstrom]

This option makes a lot of sense to add 👍

[neroist]

Another issue I have is that Nimib changes the current directory, which forces me to change it back myself (I use Prologue which searches for ./.config). I'll make an issue when/if I'm able get the wording right, and PR if I'm able to fix it.

[pietroppeter]

Looks great to me! (the only moment I paused is the echo left in blocks.nim, but I get that since we have a different type of output "prefix" we cannot reuse log and leaving the echo seems the most appropriate thing)

[pietroppeter]

Thanks for this PR, it definitely makes sense to add the flag!

This also will help the moment we want to improve logging in some way (but not doing that before someone has a clear use case or a need ;)).

@HugoGranstrom not sure if you also want to review, otherwise for me this is good to merge!

[pietroppeter]

Another issue I have is that Nimib changes the current directory, which forces me to change it back myself (I use Prologue which searches for ./.config). I'll make an issue when/if I'm able get the wording right, and PR if I'm able to fix it.

yes, there is a reason it changes directory (to access/create assets in the "build" directory, actually called homeDir, so that if I generate an image it is part of the assets that the static site will be able to see), but I can see other use cases not wanting that. I think it make sense to add a flag there (we are starting to have a bunch of them, at some point we might want to document them somewhere).

For reference, the change of directory is here:

nimib/src/nimib.nim

Line 46 in 02b5253

if nb.cfg.homeDir != "":

[pietroppeter]

...and I just realized that this feature (-d:nimibLog) should be documented somewhere (I guess in the Readme/nimib main doc), otherwise it is only for the insiders :)

[HugoGranstrom]

A bit of documentation and I'm content with merging this 😁🚀

[neroist]

looking at docsrc/config.nims I realize that nimibQuiet is probably a better & more elegant name than nimibNoLog 😅

[HugoGranstrom]

looking at docsrc/config.nims I realize that nimibQuiet is probably a better & more elegant name than nimibNoLog 😅

Oh look at that :o Yes nimibQuiet would probably be a better name for the switch

[neroist]

name changed & docs added! 👍

[pietroppeter]

haha, forgot about that nimibQuiet, I must have introduced it back then for the same purpose :)
I am a bit on the fence on the change of name since nimibNoLog was really clear in its purpose, but nimibQuiet is more playful and you both seem to approve of it.

thanks again, looks good, will edit the title (so that when we make the changelog it will be already correct) and will merge this.