Skip to content

fix: stricter input parsing and more lenient parsing of run exports from other packages #1794

fix: stricter input parsing and more lenient parsing of run exports from other packages

fix: stricter input parsing and more lenient parsing of run exports from other packages #1794

Triggered via pull request December 17, 2024 18:44
Status Success
Total duration 18m 9s
Artifacts 7

rust.yml

on: pull_request
Matrix: build
Matrix: test
Post-process
2s
Post-process
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 2 warnings
test (ubuntu-latest): src/recipe/parser/requirements.rs#L374
[clippy] reported by reviewdog 🐶 error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | Raw Output: src/recipe/parser/requirements.rs:374:9:e:error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | __END__
test (ubuntu-latest): src/recipe/parser/requirements.rs#L374
[clippy] reported by reviewdog 🐶 error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | Raw Output: src/recipe/parser/requirements.rs:374:9:e:error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | __END__
test (macos-latest): src/recipe/parser/requirements.rs#L374
[clippy] reported by reviewdog 🐶 error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | Raw Output: src/recipe/parser/requirements.rs:374:9:e:error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | __END__
test (macos-latest): src/recipe/parser/requirements.rs#L374
[clippy] reported by reviewdog 🐶 error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | Raw Output: src/recipe/parser/requirements.rs:374:9:e:error: this `if` statement can be collapsed --> src/recipe/parser/requirements.rs:374:9 | 374 | / if split_string.len() >= 2 { 375 | | if split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 376 | | let name = split_string[0]; 377 | | let version = split_string[1]; ... | 392 | | } 393 | | } | |_________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `-D clippy::collapsible-if` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::collapsible_if)]` help: collapse nested if block | 374 ~ if split_string.len() >= 2 && split_string[1].chars().all(|c| c.is_numeric() || c == '.') { 375 + let name = split_string[0]; 376 + let version = split_string[1]; 377 + let rest = split_string[2..].join(" "); 378 + let rest = if rest.is_empty() { 379 + "".to_string() 380 + } else { 381 + format!(" {}", rest) 382 + }; 383 + 384 + return Err(vec![_partialerror!( 385 + *self.span(), 386 + ErrorKind::Other, 387 + label = format!( 388 + "This match spec is ambiguous. Do you mean `{name} =={version}{rest}` or `{name} {version}.*{rest}`?" 389 + ) 390 + )]); 391 + } | __END__
test (ubuntu-latest)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Post-process
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Artifacts

Produced during runtime
Name Size
rattler-build-aarch64-apple-darwin.tar.gz
13 MB
rattler-build-aarch64-unknown-linux-musl.tar.gz
16.6 MB
rattler-build-powerpc64le-unknown-linux-gnu.tar.gz
17.4 MB
rattler-build-x86_64-apple-darwin.tar.gz
13.5 MB
rattler-build-x86_64-pc-windows-gnu.zip
19.5 MB
rattler-build-x86_64-pc-windows-msvc.zip
11.6 MB
rattler-build-x86_64-unknown-linux-musl.tar.gz
17.3 MB