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

JSON, structured logging #379

Merged
merged 7 commits into from
Jan 13, 2025
Merged

JSON, structured logging #379

merged 7 commits into from
Jan 13, 2025

Conversation

bauersimon
Copy link
Member

No description provided.

@bauersimon bauersimon added the refactor Make code cleaner/better label Jan 9, 2025
@bauersimon bauersimon self-assigned this Jan 9, 2025
@bauersimon bauersimon force-pushed the logging branch 5 times, most recently from b59357e to 7f5ed18 Compare January 10, 2025 08:29
@bauersimon
Copy link
Member Author

bauersimon commented Jan 10, 2025

todo

  • stdout ALL
  • query responses UUID to connect with the query

@bauersimon bauersimon force-pushed the logging branch 4 times, most recently from 1345026 to be83b56 Compare January 10, 2025 14:12
@bauersimon bauersimon marked this pull request as ready for review January 10, 2025 14:21
log/logger.go Outdated Show resolved Hide resolved
cmd/eval-dev-quality/cmd/evaluate.go Show resolved Hide resolved
cmd/eval-dev-quality/cmd/evaluate_test.go Show resolved Hide resolved
log/logger.go Show resolved Hide resolved
@bauersimon bauersimon force-pushed the logging branch 2 times, most recently from 47100c3 to 7c898f1 Compare January 13, 2025 10:24
@bauersimon bauersimon merged commit 1e3e5f8 into main Jan 13, 2025
5 checks passed
@bauersimon bauersimon deleted the logging branch January 13, 2025 12:11
@bauersimon bauersimon mentioned this pull request Jan 13, 2025
88 tasks
@zimmski zimmski added this to the v1.0.0 milestone Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Make code cleaner/better
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants