Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

chore: add version print #227

Merged
merged 6 commits into from
Sep 6, 2023
Merged

Conversation

ZuLiangWang
Copy link
Contributor

Which issue does this PR close?

Closes #

Rationale for this change

Add tool to output version.

What changes are included in this PR?

Are there any user-facing changes?

None.

How does this change test

manual testing.
截屏2023-09-05 17 01 37

cmd/meta/main.go Outdated Show resolved Hide resolved
cmd/meta/main.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #227 (6d6341b) into main (7f1354e) will increase coverage by 21.39%.
Report is 30 commits behind head on main.
The diff coverage is 64.24%.

@@             Coverage Diff             @@
##             main     #227       +/-   ##
===========================================
+ Coverage   39.66%   61.06%   +21.39%     
===========================================
  Files          34       43        +9     
  Lines        4213     5339     +1126     
===========================================
+ Hits         1671     3260     +1589     
+ Misses       2317     1677      -640     
- Partials      225      402      +177     
Files Changed Coverage Δ
pkg/coderr/code.go 0.00% <ø> (ø)
server/cluster/metadata/types.go 0.00% <0.00%> (ø)
server/coordinator/procedure/util.go 0.00% <ø> (ø)
server/storage/meta.go 0.00% <ø> (ø)
server/storage/storage_impl.go 57.21% <0.00%> (-5.10%) ⬇️
server/member/watch_leader.go 40.62% <20.00%> (-2.71%) ⬇️
server/cluster/manager.go 38.96% <23.18%> (ø)
server/coordinator/factory.go 76.68% <27.27%> (+72.22%) ⬆️
.../operation/transferleader/batch_transfer_leader.go 33.80% <33.80%> (ø)
server/member/member.go 47.70% <47.82%> (-1.02%) ⬇️
... and 21 more

... and 1 file with indirect coverage changes

server/config/config.go Outdated Show resolved Hide resolved
cmd/meta/main.go Outdated Show resolved Hide resolved
jiacai2050
jiacai2050 previously approved these changes Sep 6, 2023
Copy link
Contributor

@jiacai2050 jiacai2050 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ZuLiangWang ZuLiangWang merged commit 7624f6f into apache:main Sep 6, 2023
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants