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

Logger system rewrite #59

Open
Dragovorn opened this issue Jul 23, 2023 · 2 comments
Open

Logger system rewrite #59

Dragovorn opened this issue Jul 23, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request library Something that isn't directly in the Winry codebase
Milestone

Comments

@Dragovorn
Copy link
Member

No need to rewrite the base of the logger itself, just add a heavier emphasis to having a logger factory, because that's where i want all logger configuration to be centralized so that injected loggers can all move together.

@Dragovorn Dragovorn added the enhancement New feature or request label Jul 23, 2023
@Dragovorn Dragovorn added this to the 1.0.0 milestone Jul 23, 2023
@Dragovorn Dragovorn self-assigned this Jul 23, 2023
@Dragovorn
Copy link
Member Author

Did something basic, added the ability to add logger factories in your bootstrap annotation, this will probably be the logging functionality stop-gap for awhile

@Dragovorn
Copy link
Member Author

This is likely going to be an Araragi change, but I'll keep it here for tracking purposes.

@Dragovorn Dragovorn added the library Something that isn't directly in the Winry codebase label Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request library Something that isn't directly in the Winry codebase
Projects
None yet
Development

No branches or pull requests

1 participant