Closed hackerwins closed 3 weeks ago
[!WARNING]
Review failed
The pull request is closed.
The changes primarily involve simplifying the version information retrieval in the cmd/yorkie/version.go
file and updating a log message in server/backend/backend.go
. Specifically, the settingByKey
function and a relative constant were removed for a more straightforward approach to print the version and build information. Meanwhile, in backend.go
, the log message was refined to correctly display the database reference instead of the RPC reference during backend creation.
File | Change Summary |
---|---|
cmd/yorkie/version.go |
Removed settingByKey function and commitRevisionKey constant for simplified version info retrieval. |
server/backend/backend.go |
Modified the log message in New() function to correctly reference db instead of rpc . |
In code’s forest, changes bloom,
Simplified the version's room.
Yorkie's build now clear and bright,
Backend logs with names just right.
Happy bits and bytes now play,
As Rabbit cheers, in code's ballet.
🌼💻✨
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?
Attention: Patch coverage is 0%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 50.74%. Comparing base (
47ae9d4
) to head (08d4e97
).
Files | Patch % | Lines |
---|---|---|
server/backend/backend.go | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What this PR does / why we need it:
Remove commit revision in version command
The following error occurs when running
$ yorkie version
.The cause was not found, but since revision wasn't strictly necessary, this commit removes
revision
fromversion
command.Starting with 0.4.18, BrewTestBot started automatically deploying to Homebrew. It is assumed that the error occurs after this point.
Which issue(s) this PR fixes:
Related to #870
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation:
Checklist:
Summary by CodeRabbit
Refactor
Style