Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@
/src/api/data_types/snapshots.rs @getsentry/emerge-tools @getsentry/owners-sentry-cli
/src/commands/build @getsentry/emerge-tools @getsentry/owners-sentry-cli
/src/utils/build @getsentry/emerge-tools @getsentry/owners-sentry-cli
/src/commands/snapshots @getsentry/emerge-tools @getsentry/owners-sentry-cli
/src/utils/odiff @getsentry/emerge-tools @getsentry/owners-sentry-cli
/tests/integration/build @getsentry/emerge-tools @getsentry/owners-sentry-cli

# Files without codeowner protection
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### Features

- (snapshots) Add `snapshots diff` command for locally comparing directories of PNG snapshot images using odiff ([#3306](https://github.com/getsentry/sentry-cli/pull/3306))

### Fixes

- Improve error message when organization slug is missing from config ([#3311](https://github.com/getsentry/sentry-cli/pull/3311))
Expand Down
13 changes: 11 additions & 2 deletions src/commands/build/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,11 @@ use clap::{ArgMatches, Command};
use crate::utils::args::ArgExt as _;

pub mod download;
pub mod snapshots;
pub mod upload;

macro_rules! each_subcommand {
($mac:ident) => {
$mac!(download);
$mac!(snapshots);
$mac!(upload);
};
}
Expand All @@ -31,6 +29,11 @@ pub fn make_command(mut command: Command) -> Command {
.org_arg()
.project_arg(true);
each_subcommand!(add_subcommand);
command = command.subcommand(
crate::commands::snapshots::upload::make_command(Command::new("snapshots"))
.hide(true)
.about("[DEPRECATED] Use `sentry-cli snapshots upload` instead."),
);
command
}

Expand All @@ -45,5 +48,11 @@ pub fn execute(matches: &ArgMatches) -> Result<()> {
}};
}
each_subcommand!(execute_subcommand);
if let Some(sub_matches) = matches.subcommand_matches("snapshots") {
log::warn!(
"`sentry-cli build snapshots` is deprecated. Use `sentry-cli snapshots upload` instead."
);
return crate::commands::snapshots::upload::execute(sub_matches);
}
unreachable!();
}
2 changes: 2 additions & 0 deletions src/commands/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ mod releases;
mod repos;
mod send_envelope;
mod send_event;
mod snapshots;
mod sourcemaps;
#[cfg(not(feature = "managed"))]
mod uninstall;
Expand Down Expand Up @@ -70,6 +71,7 @@ macro_rules! each_subcommand {
$mac!(repos);
$mac!(send_event);
$mac!(send_envelope);
$mac!(snapshots);
$mac!(sourcemaps);
$mac!(dart_symbol_map);
#[cfg(not(feature = "managed"))]
Expand Down
Loading
Loading