Skip to content

Commit dfd89eb

Browse files
authored
Fix unprocessed packages showing as complete (#1517)
Closes #1516.
1 parent 8a308ae commit dfd89eb

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
88

99
## Unreleased
1010

11+
### Fixed
12+
13+
- `phylum package` subcommand showing unprocessed packages as complete
14+
1115
## 7.1.1 - 2024-10-09
1216

1317
### Changed

cli/src/commands/packages.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,22 +35,22 @@ pub async fn handle_get_package(api: &PhylumApi, matches: &clap::ArgMatches) ->
3535
};
3636

3737
match resp {
38-
PackageSubmitResponse::AlreadyProcessed(mut resp) => {
38+
PackageSubmitResponse::AlreadyProcessed(mut resp) if resp.complete => {
3939
let filter = matches.get_one::<String>("filter").and_then(|v| Filter::from_str(v).ok());
4040
if let Some(filter) = filter {
4141
resp.filter(&filter);
4242
}
4343

4444
resp.write_stdout(pretty_print);
4545
},
46-
PackageSubmitResponse::AlreadySubmitted => {
46+
PackageSubmitResponse::New => {
4747
print_user_warning!(
48-
"Package is still processing. Please check back later for results."
48+
"Thank you for submitting this package. Please check back later for results."
4949
);
5050
},
51-
PackageSubmitResponse::New => {
51+
_ => {
5252
print_user_warning!(
53-
"Thank you for submitting this package. Please check back later for results."
53+
"Package is still processing. Please check back later for results."
5454
);
5555
},
5656
}

0 commit comments

Comments
 (0)