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

Debug logging #157

Merged
merged 2 commits into from
Feb 4, 2024
Merged

Debug logging #157

merged 2 commits into from
Feb 4, 2024

Conversation

braydonk
Copy link
Collaborator

@braydonk braydonk commented Feb 3, 2024

This PR adds debug logging. Debug logging can be enabled through flags to the yamlfmt command, and will tell you in very noisy detail about things that are going on. Debug groups are separated into codes, so that only certain debug logs can be enabled if you are debugging a specific problem.

I also fixed a bug I introduced in the last config PR with the -conf flag. It is fixed before ever going out in a release, so it will only affect someone who installed from that exact commit before I fix it here.

This PR adds debug logging. Debug logging can be enabled through flags
to the yamlfmt command, and will tell you in very noisy detail about
things that are going on. Debug groups are separated into codes, so that
only certain debug logs can be enabled if you are debugging a specific
problem.

I also fixed a bug I introduced in the last config PR with the `-conf`
flag. It is fixed before ever going out in a release, so it will only
affect someone who installed from that exact commit before I fix it
here.
The integration test files got accidentally formatted by some local
testing, this sets them back to what they should be. Also adds a new
flag that that disables usage of configuration file from system home.
@braydonk braydonk merged commit f33dc61 into google:main Feb 4, 2024
5 checks passed
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

Successfully merging this pull request may close these issues.

1 participant