search

unyt-org / uix

The UIX Framework
https://uix.unyt.org
MIT License
54 stars 3 forks source link

Unify Error Logging and Migrate Base Project Initialization to Deno 2.0 #157

Closed asbng closed 2 months ago

asbng commented 2 months ago

Unified error messages to be presented using handleError(). The deprecated Deno.run was replaced in favor of Deno.Command which is the way to move forward with Deno 2.0.

Fixes #145 .

benStre commented 2 months ago

Not directly related to this PR , but do we always want to explicitly pass a logger? Can't we just set a default logger?

asbng commented 2 months ago

Feel free to. I do, however, think that it doesn't hurt explicitly printing to the logger that the error originates from. Explicit over implicit, as long as productivity doesn't suffer (and specifying logger was a no-brainer in this PR's code). It is reasonable to have a default fallback where a logger is not at hand though.