Allow to configure the writer to something other than stdout

[lowlighter]

It seems like currently it's hardcoded to stdout:

progress/mod.ts

Line 57 in bcc5d0d

private writer = Deno.stdout.writable.getWriter();

But it'd be nice to be able to set it to stderr too

The reasoning is that for cli, it's not uncommon to depends on stdout returns when using the shell syntax (like $()), but currently this messes the output

deno --allow-run=$(deno task install-and-return-path) run mod.ts

Kind similar behavior like curl/wget uses for progress bar.
Actually it may be a better default to use stderr, but it may be a breaking change

[fuxingZhang]

Today I'm going to release a new version to support stderr.
I'd also be more than happy to merge your pull request.

[fuxingZhang]

Landed on v1.5.0

progress/examples/stderr.ts

Line 9 in 45e626d

output: Deno.stderr,

[fuxingZhang]

Initially, I was using writeAllSync, but on December 27, 2023, I discovered that writeAllSync was deprecated, and the documentation recommended using the stream API.

https://deno.land/[email protected]/streams/write_all.ts?s=writeAll

(will be removed after 1.0.0) Use WritableStream, ReadableStream.from, and pipeTo()

As a result, I released version 1.4.1 (Remove deprecated writeAllSync, Use WritableStream instead).

However, today I found that both writeAll and writeAllSync are now available again. I checked the source code and issues, and found:

denoland/std#4120
denoland/std#4128

I believe writeAll is simpler. Therefore, I changed Deno.stdout.writable.getWriter() to writeAll. Since I am currently using the promise style, so I changed to use writeAll.

Loaded on v1.5.1.

[fuxingZhang]

I forgot to change the version in the deno.json, so I force pushed the changes again, and it's working fine now.

[lowlighter]

Nice ! Thanks a lot for being so quick !