Skip to content

Commit

Permalink
Remove the unstage command
Browse files Browse the repository at this point in the history
Remove the unstage command because it is simply a proxy command and
doesn't provide any real value. Plus it add noise to the command surface
area making it more overwhelming for new users.

[changelog]
removed: the unstage command

<!-- ps-id: 01672bec-62a4-4f7e-9e31-408674fe8c74 -->
  • Loading branch information
drewdeponte committed Oct 20, 2023
1 parent 385a6d7 commit 02a0b36
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 40 deletions.
9 changes: 0 additions & 9 deletions src/cli.rs
Original file line number Diff line number Diff line change
Expand Up @@ -88,12 +88,6 @@ pub struct AmendPatchOpts {
pub no_edit: bool,
}

#[derive(Debug, Args)]
pub struct UnstageCmdOpts {
/// specific files to unstage changes for, leave blank for all staged files
pub files: Vec<String>,
}

#[derive(Debug, Args)]
pub struct BackupStackCmdOpts {
pub branch_name: String,
Expand Down Expand Up @@ -277,9 +271,6 @@ stack.
/// display a log of integrated patches
Log,

/// unstage currently staged changes
Unstage(UnstageCmdOpts),

/// (f) - Fetch state from remote and display upstream patches
#[command(name = "fetch", alias = "f")]
Fetch,
Expand Down
1 change: 0 additions & 1 deletion src/commands/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,4 @@ pub mod request_review_branch;
pub mod show;
pub mod status;
pub mod sync;
pub mod unstage;
pub mod utils;
11 changes: 0 additions & 11 deletions src/commands/unstage.rs

This file was deleted.

1 change: 0 additions & 1 deletion src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,5 @@ pub use ps::public::request_review_branch::{request_review_branch, RequestReview
pub use ps::public::show::show;
pub use ps::public::status::status;
pub use ps::public::sync::{sync, SyncError};
pub use ps::public::unstage::unstage;
pub use ps::public::upstream_patches::upstream_patches;
pub use ps::public::verify_isolation::{verify_isolation, VerifyIsolationError};
1 change: 0 additions & 1 deletion src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ fn main() {
cli::Command::AmendPatch(opts) => commands::amend_patch::amend_patch(opts.no_edit),
cli::Command::Status => commands::status::status(),
cli::Command::Log => commands::log::log(),
cli::Command::Unstage(opts) => commands::unstage::unstage(opts.files),
cli::Command::Fetch => commands::fetch::fetch(cli.color),
#[cfg(feature = "backup_cmd")]
cli::Command::BackupStack(opts) => commands::backup_stack::backup_stack(opts.branch_name),
Expand Down
1 change: 0 additions & 1 deletion src/ps/public/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,5 @@ pub mod request_review_branch;
pub mod show;
pub mod status;
pub mod sync;
pub mod unstage;
pub mod upstream_patches;
pub mod verify_isolation;
16 changes: 0 additions & 16 deletions src/ps/public/unstage.rs

This file was deleted.

0 comments on commit 02a0b36

Please sign in to comment.