Open janezpodhostnik opened 2 hours ago
The pull request introduces changes in the bootstrap/bootstrap.go
file, primarily renaming the startEngine
function to StartEngine
, thus enhancing its visibility. Logging improvements are made to include a duration metric for the engine startup process. Additionally, the go.mod
files for both the main module and the integration module are updated to reflect a new version of the github.com/rs/zerolog
dependency, while maintaining the existing Go version and other dependencies.
File(s) | Change Summary |
---|---|
bootstrap/bootstrap.go | Renamed startEngine to StartEngine , updated logging to include duration metric, and adjusted logger usage. |
go.mod | Updated github.com/rs/zerolog dependency from v1.31.0 to v1.33.0 . |
tests/go.mod | Updated github.com/rs/zerolog dependency from v1.31.0 to v1.33.0 . |
sequenceDiagram
participant A as StartEventIngestion
participant B as StartTraceDownloader
participant C as StartEngine
A->>C: Call StartEngine
B->>C: Call StartEngine
C->>C: Initialize engine
C->>C: Log startup duration
C-->>A: Return
C-->>B: Return
🐇 In the code where bunnies hop,
Functions renamed, they never stop.
Logs now sing with metrics bright,
Engines roar with newfound might!
Dependencies updated, all is well,
In the land of code, we dance and dwell! 🎉
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@janezpodhostnik I think you also need to run:
$ cd tests/
$ go mod tidy
Description
Just cleaning up a bit as I get familiar with the codebase.
github.com/rs/zerolog v1.33.0
is required forlogger.With().Type("engine", engine)
.For contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit
zerolog
dependency to the latest version for improved logging capabilities.