-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Remove immutable deploy #28399
Remove immutable deploy #28399
Conversation
1c98d3a
to
17629eb
Compare
@@ -679,26 +672,6 @@ pub fn parse_command( | |||
} | |||
("upgrade-nonce-account", Some(matches)) => parse_upgrade_nonce_account(matches), | |||
// Program Deployment | |||
("deploy", Some(matches)) => { |
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.
What do you think about leaving the SubCommand
and using this match case to return an Error exit pointing users at solana program deploy
?
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.
Might be good for user experience. I've added back deploy in 2a5b910
2a5b910
to
e51470b
Compare
Pull request has been modified.
* remove Deploy option from cli * remove solana deploy from comments * introduce dummy deploy command to improve user experience
* remove Deploy option from cli * remove solana deploy from comments * introduce dummy deploy command to improve user experience
Problem
This PR is to remove
solana deploy
command, see #27834This PR depends on:
#28398 (merged)
#28448 (merged)
Summary of Changes
deploy
subcommandsolana deploy
command from code documentation