Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logging #86

Open
bencyoung-Fignum opened this issue May 26, 2023 · 1 comment
Open

Logging #86

bencyoung-Fignum opened this issue May 26, 2023 · 1 comment

Comments

@bencyoung-Fignum
Copy link

Hi,

It would be great if we could add logging for certain situations (on request, , missing secrets, if JSON parsing fails etc).

Would you accept a PR?
If so would you want it with a simple Action callback in the options, or would you be happy with a way to bind an ILogging implementation somehow?

Thanks

@Kralizek
Copy link
Owner

I'm currently too busy to work on the library but I welcome PRs under the understanding that I cannot promise when I will have time to look at it.

Anyway, you can use closures to inject a logger in methods that are used for configuration. I know it's not enough but it could be a starter to help you troubleshooting your issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants