holm/nushell
1
1
Fork 0
forked from extern/nushell
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bump calamine and fix clippy (#10314)

Browse Source 
Upgrade calamine to 0.22
Reduces one potential dependency duplication

Supersedes #10305

Includes fixes for clippy lints as API changed to return owned data.
This commit is contained in:
Stefan Holderbach
2023-09-11 12:39:52 +02:00
committed by GitHub
parent baa50ec9b2
commit eddff46155
4 changed files with 8 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
crates/nu-command/src/formats/from/ods.rs
View File

@ -128,7 +128,7 @@ fn from_ods(
let mut dict = IndexMap::new();
let mut sheet_names = ods.sheet_names().to_owned();
let mut sheet_names = ods.sheet_names();
if !sel_sheets.is_empty() {
sheet_names.retain(|e| sel_sheets.contains(e));
}

2
crates/nu-command/src/formats/from/xlsx.rs
View File

@ -127,7 +127,7 @@ fn from_xlsx(
let mut dict = IndexMap::new();
let mut sheet_names = xlsx.sheet_names().to_owned();
let mut sheet_names = xlsx.sheet_names();
if !sel_sheets.is_empty() {
sheet_names.retain(|e| sel_sheets.contains(e));
}