fix: remove duplicate manifest name in publish command

This commit is contained in:
daimond113 2024-10-12 18:59:57 +02:00
parent b6459623b7
commit 43a8d6272a
No known key found for this signature in database
GPG key ID: 3A8ECE51328B513C
2 changed files with 3 additions and 4 deletions

View file

@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Fixed ### Fixed
- Fix `self-upgrade` overwriting its own binary by @daimond113 - Fix `self-upgrade` overwriting its own binary by @daimond113
- Allow use of Luau packages in `execute` command by @daimond113 - Allow use of Luau packages in `execute` command by @daimond113
- Remove duplicated manifest file name in `publish` command by @daimond113
## [0.5.0-rc.4] - 2024-10-12 ## [0.5.0-rc.4] - 2024-10-12
### Added ### Added

View file

@ -107,8 +107,6 @@ impl PublishCommand {
}; };
if !manifest.includes.insert(MANIFEST_FILE_NAME.to_string()) { if !manifest.includes.insert(MANIFEST_FILE_NAME.to_string()) {
display_includes.push(MANIFEST_FILE_NAME.to_string());
println!( println!(
"{}: {MANIFEST_FILE_NAME} was not in includes, adding it", "{}: {MANIFEST_FILE_NAME} was not in includes, adding it",
"warn".yellow().bold() "warn".yellow().bold()
@ -224,7 +222,7 @@ impl PublishCommand {
anyhow::bail!("included file {included_name} does not exist"); anyhow::bail!("included file {included_name} does not exist");
} }
// it'll be included later, with our mut modifications // it's already included, and guaranteed to be a file
if included_name.eq_ignore_ascii_case(MANIFEST_FILE_NAME) { if included_name.eq_ignore_ascii_case(MANIFEST_FILE_NAME) {
continue; continue;
} }