blinklabs-io / bursa

Programmatic Cardano Wallet
Apache License 2.0
5 stars 2 forks source link

feat: add cobra cli library #89

Closed oversize closed 3 months ago

oversize commented 3 months ago

This PR adds cobra as the cli parsing library. https://github.com/blinklabs-io/bursa/issues/63

wolf31o2 commented 3 months ago

Looks like you have conflicts with main.

oversize commented 3 months ago

Looks like you have conflicts with main.

Oh snap, yes. I have merged these changes now and updated the commits. I also just saw that cobra still provided its default "generate shell completion" script, which i now have disabled.

oversize commented 3 months ago

Back out the changes to remove the internal/logging module. I'm fine with the introduction of slog for console logging, but we need the access logs in the API to stay.

I have removed the console logging again in favor of internal/logging. I'll look into gin and how to have access logs with log/slog so we dont lose these when switching to it.