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

cmd: remove deprecated commands #551

Merged
merged 1 commit into from
May 17, 2022
Merged

Conversation

dB2510
Copy link
Contributor

@dB2510 dB2510 commented May 17, 2022

Refactors create command, removing deprecated cmds

category: refactor
ticket: #550

@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #551 (fedd36f) into main (dd4336b) will increase coverage by 0.36%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #551      +/-   ##
==========================================
+ Coverage   54.73%   55.09%   +0.36%     
==========================================
  Files          92       91       -1     
  Lines        8589     8538      -51     
==========================================
+ Hits         4701     4704       +3     
+ Misses       3260     3208      -52     
+ Partials      628      626       -2     
Impacted Files Coverage Δ
cmd/cmd.go 75.64% <0.00%> (+3.68%) ⬆️
cmd/createcluster.go 54.08% <100.00%> (+2.45%) ⬆️
core/qbft/qbft.go 82.13% <0.00%> (ø)
app/app.go 64.20% <0.00%> (+0.88%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd4336b...fedd36f. Read the comment docs.

@dB2510 dB2510 linked an issue May 17, 2022 that may be closed by this pull request
Copy link
Contributor

@xenowits xenowits left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -43,17 +43,13 @@ func New() *cobra.Command {
return newRootCmd(
newVersionCmd(runVersionCmd),
newEnrCmd(runNewENR),
// TODO(dhruv): remove genp2pkey command once charon-docker-compose and docs are updated
Copy link
Contributor

Choose a reason for hiding this comment

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

are the docs updated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dB2510 dB2510 changed the title cmd: refactor create command cmd: remove deprecated commands May 17, 2022
@dB2510 dB2510 added the merge when ready Indicates bulldozer bot may merge when all checks pass label May 17, 2022
@obol-bulldozer obol-bulldozer bot merged commit 8f073a8 into main May 17, 2022
@obol-bulldozer obol-bulldozer bot deleted the dhruv/refactorcreate branch May 17, 2022 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when ready Indicates bulldozer bot may merge when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cmd: remove deprecated command charon create-cluster
3 participants