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.

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 account

Jump to bottom

Use slog #235

Closed
gligneul opened this issue Jan 10, 2024 · 1 comment
Closed

Use slog #235

gligneul opened this issue Jan 10, 2024 · 1 comment
Assignees
@torives
Labels
#feat:go-supervisor Feature: Go supervisor
Milestone
1.4.0

Comments

@gligneul
Copy link
Contributor

📚 Context

This issue aims to refactor the Go code to use the log/slog package instead of a custom-made one. Then, we can use the tint package to make the log pretty in the local environment. By default, the log handler should be JSON or something that is easy to parse automatically in a cloud environment.
@gligneul gligneul added #feat:go-supervisor Feature: Go supervisor ! High priority labels Jan 10, 2024
@gligneul gligneul added this to the 1.3.0 milestone Jan 10, 2024
@gligneul gligneul moved this to 📋 Backlog in Node Unit Jan 10, 2024
@gligneul gligneul removed the ! High priority label Jan 15, 2024
@gligneul gligneul moved this from 📋 Backlog to 🏗 In progress in Node Unit Feb 5, 2024
@gligneul gligneul modified the milestones: 1.3.0, 1.4.0 Feb 9, 2024
@torives torives mentioned this issue Feb 9, 2024
Closed
@torives torives moved this from 🏗 In progress to 👀 In review in Node Unit Feb 12, 2024
@torives torives moved this from 👀 In review to 🏗 In progress in Node Unit Mar 8, 2024
@gligneul
Copy link
Contributor Author

This will be replaced by #325

@gligneul gligneul moved this from 🏗 In progress to ✅ Done in Node Unit Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@torives torives

Labels
#feat:go-supervisor Feature: Go supervisor
Projects
Node Unit
Archived in project
Milestone
1.4.0
Development

Successfully merging a pull request may close this issue.

feat: replace log with slog cartesi/rollups-node
2 participants
@torives @gligneul