-
-
Notifications
You must be signed in to change notification settings - Fork 351
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
Drop Legacy Logger from Codebase #4171
Drop Legacy Logger from Codebase #4171
Conversation
fb0bfe5
to
d8380bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copy-pasted 2 errors from CI to hopefully save you bit of time 😄
0e7e71c
to
552885a
Compare
I am pretty sure I broke opentelemetry support, so I have yet to fix that :) |
427d38f
to
bfd4629
Compare
I think I fixed the opentelemetry stuff, hopefully only minor mistakes. @soulomoon assured me, we can check that opentelemetry still works as expected by looking at the benchmarks :) So, we still have to check the benchmarks and report the results here. |
1515483
to
5bc9fd4
Compare
Nothing escapes your eagle-eyed review 😅 Thanks, should be all updated now. |
Move ghcide completely to colog-logging style. Move plugins that were relying on `ideLogger` to colog style logging. Move opentelemetry to colog-logging style. This allows us to drop legacy code and unify the logging experience in HLS. We add a bunch of new Log constructors at various locations that aim to be identical to their previous `Logger` statements.
5bc9fd4
to
5bc5251
Compare
Move ghcide completely to colog-logging style.
Move plugins that were relying on
ideLogger
to colog style logging.Move opentelemetry to colog-logging style.
This allows us to drop legacy code and unify the logging experience in
HLS.
We add a bunch of new Log constructors at various locations that aim to
be identical to their previous
Logger
statements.Just a bit of busy work. It started out as migrating
ekg
to colog style logging.Now only the telemetry logger usesFixed.Logger
.