8000 Fix formatting of log message args in verbose mode by kvch · Pull Request #817 · xataio/pgroll · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix formatting of log message args in verbose mode #817

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.

Sign up for GitHub

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 8000 account

Jump to bottom
Merged
merged 1 commit into from
May 7, 2025
Merged

Fix formatting of log message args in verbose mode #817

merged 1 commit into from
May 7, 2025

Conversation

kvch
Copy link
Contributor
@kvch kvch commented May 7, 2025 •
edited
Loading

This PR fixes the format of log message arguments. Previously, the arguments did not show up as key values pairs, but as a single list of args with an error message:

2025-05-07 14:20:41 INFO  starting operation
                      └ ERROR: key_without_value: [operation create_table name customers columns [id name credit_card] comment <nil> constraints []]
2025-05-07 14:20:41 INFO  starting operation
                      └ ERROR: key_without_value: [operation create_table name bills columns [id date quantity] comment <nil> constraints []]
2025-05-07 14:20:41 INFO  starting operation
                      └ ERROR: key_without_value: [operation create_table name sellers columns [name zip description] comment <nil> constraints []]

@github-actions github-actions bot temporarily deployed to Docs Preview May 7, 2025 13:32 Inactive
@kvch kvch requested a review from andrew-farries May 7, 2025 13:33
@kvch kvch merged commit fcf713e into xataio:main May 7, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrew-farries andrew-farries andrew-farries approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kvch @andrew-farries
0