buildpacks / libcnb

A non-opinionated language binding for the Cloud Native Buildpack Buildpack and Extension specifications
Apache License 2.0
31 stars 13 forks source link

Move formatters out of main package #146

Closed dmikusa closed 1 year ago

dmikusa commented 2 years ago

Thoughts on this one?


The primary purpose of this PR is to move the formatters that were only used for debug logging out of the main package.

Doing this has a few other side effects that are helpful as well:

Signed-off-by: Daniel Mikusa dmikusa@vmware.com

samj1912 commented 1 year ago

@dmikusa - I made some minor changes so that the default formatter (plain formatter) is internal only since it takes no args. I changed the interface for the content writer a bit to reuse the formatter/debug writer and had the write method take in both the title and the path.