Skip to content

add dump-external links command (#168) #483

add dump-external links command (#168)

add dump-external links command (#168) #483

Triggered via push November 29, 2023 14:46
Status Success
Total duration 2m 27s
Artifacts

test.yml

on: push
Matrix: Build and test
Fit to window
Zoom out
Zoom in

Annotations

25 warnings
manual check for common ascii range: src/urls.rs#L17
warning: manual check for common ascii range --> src/urls.rs:17:9 | 17 | if !matches!(first_char, b'a'..=b'z' | b'A'..=b'Z') { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `first_char.is_ascii_alphabetic()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_is_ascii_check = note: `#[warn(clippy::manual_is_ascii_check)]` on by default
this expression creates a reference which is immediately dereferenced by the compiler: src/collector.rs#L97
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/collector.rs:97:54 | 97 | let mut href = BumpString::from_str_in(path, &arena); | ^^^^^^ help: change this to: `arena` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
unnecessary closure used to substitute value for `Option::None`: src/collector.rs#L89
warning: unnecessary closure used to substitute value for `Option::None` --> src/collector.rs:89:24 | 89 | let qs_start = used_link | ________________________^ 90 | | .href 91 | | .0 92 | | .find(&['?', '#'][..]) 93 | | .unwrap_or_else(|| used_link.href.0.len()); | |______________----------------------------------------^ | | | help: use `unwrap_or(..)` instead: `unwrap_or(used_link.href.0.len())` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations = note: `#[warn(clippy::unnecessary_lazy_evaluations)]` on by default
this expression creates a reference which is immediately dereferenced by the compiler: src/collector.rs#L85
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/collector.rs:85:29 | 85 | if is_external_link(&used_link.href.0.as_bytes()) { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `used_link.href.0.as_bytes()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `#[warn(clippy::needless_borrow)]` on by default
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/collector.rs#L72
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/collector.rs:72:62 | 72 | LinkState::Undefined(links2) => links.extend(links2.into_iter()), | ^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `links2` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/79e9716c980570bfd1f666e3b16ac583f0168962/library/core/src/iter/traits/collect.rs:371:18 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
Build and test (linux)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Build and test (linux)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (linux)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (linux)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (linux)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
manual check for common ascii range: src/urls.rs#L17
warning: manual check for common ascii range --> src/urls.rs:17:9 | 17 | if !matches!(first_char, b'a'..=b'z' | b'A'..=b'Z') { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `first_char.is_ascii_alphabetic()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_is_ascii_check = note: `#[warn(clippy::manual_is_ascii_check)]` on by default
this expression creates a reference which is immediately dereferenced by the compiler: src/collector.rs#L97
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/collector.rs:97:54 | 97 | let mut href = BumpString::from_str_in(path, &arena); | ^^^^^^ help: change this to: `arena` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
unnecessary closure used to substitute value for `Option::None`: src/collector.rs#L89
warning: unnecessary closure used to substitute value for `Option::None` --> src/collector.rs:89:24 | 89 | let qs_start = used_link | ________________________^ 90 | | .href 91 | | .0 92 | | .find(&['?', '#'][..]) 93 | | .unwrap_or_else(|| used_link.href.0.len()); | |______________----------------------------------------^ | | | help: use `unwrap_or(..)` instead: `unwrap_or(used_link.href.0.len())` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_lazy_evaluations = note: `#[warn(clippy::unnecessary_lazy_evaluations)]` on by default
this expression creates a reference which is immediately dereferenced by the compiler: src/collector.rs#L85
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/collector.rs:85:29 | 85 | if is_external_link(&used_link.href.0.as_bytes()) { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `used_link.href.0.as_bytes()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `#[warn(clippy::needless_borrow)]` on by default
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/collector.rs#L72
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/collector.rs:72:62 | 72 | LinkState::Undefined(links2) => links.extend(links2.into_iter()), | ^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `links2` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/79e9716c980570bfd1f666e3b16ac583f0168962/library/core/src/iter/traits/collect.rs:371:18 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
Build and test (macos)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Build and test (macos)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (macos)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (macos)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (macos)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (windows)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Build and test (windows)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (windows)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (windows)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Build and test (windows)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/