Skip to content

feat: generate sitemap and robots.txt #59

feat: generate sitemap and robots.txt

feat: generate sitemap and robots.txt #59

Triggered via pull request June 16, 2024 21:07
Status Success
Total duration 1m 15s
Artifacts

rust.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
this function has too many arguments (8/7): src/post.rs#L103
warning: this function has too many arguments (8/7) --> src/post.rs:103:5 | 103 | / pub fn new( 104 | | front_matter: FrontMatter, 105 | | excerpt: Option<String>, 106 | | content: String, ... | 111 | | ancestor_directories_names: Vec<String>, 112 | | ) -> Self { | |_____________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments = note: `#[warn(clippy::too_many_arguments)]` on by default
comparison to empty slice: src/content.rs#L201
warning: comparison to empty slice --> src/content.rs:201:20 | 201 | if dir_index != "" { | ^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!dir_index.is_empty()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#comparison_to_empty = note: `#[warn(clippy::comparison_to_empty)]` on by default
this expression creates a reference which is immediately dereferenced by the compiler: src/content.rs#L68
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/content.rs:68:9 | 68 | &site.settings.website_title.as_str(), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `site.settings.website_title.as_str()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
this expression creates a reference which is immediately dereferenced by the compiler: src/content.rs#L67
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/content.rs:67:9 | 67 | &site.settings.author.as_str(), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `site.settings.author.as_str()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
this expression creates a reference which is immediately dereferenced by the compiler: src/content.rs#L66
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/content.rs:66:9 | 66 | &site.settings.base_url.as_str(), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `site.settings.base_url.as_str()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
this expression creates a reference which is immediately dereferenced by the compiler: src/content.rs#L60
warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/content.rs:60:9 | 60 | &site.settings.base_url.as_str(), | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `site.settings.base_url.as_str()` | = 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
assigning the result of `ToOwned::to_owned()` may be inefficient: src/config/markup.rs#L174
warning: assigning the result of `ToOwned::to_owned()` may be inefficient --> src/config/markup.rs:174:13 | 174 | target = "target=\"_blank\" ".to_owned(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `clone_into()`: `"target=\"_blank\" ".clone_into(&mut target)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#assigning_clones = note: `#[warn(clippy::assigning_clones)]` on by default
this `if` statement can be collapsed: src/content.rs#L201
warning: this `if` statement can be collapsed --> src/content.rs:201:17 | 201 | / if dir_index != "" { 202 | | if site 203 | | .settings 204 | | .create_index_for ... | 210 | | } 211 | | } | |_________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#collapsible_if = note: `#[warn(clippy::collapsible_if)]` on by default help: collapse nested if block | 201 ~ if dir_index != "" && site 202 + .settings 203 + .create_index_for 204 + .contains(&dir_index.to_string()) { 205 + let key = dir_index.to_string(); 206 + let posts = indexes_to_create.entry(key).or_default(); 207 + posts.push(post); 208 + } |
build
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions-rs/clippy-check@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, 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/