Order RSS by date ASC and fix updated_at date #41
Annotations
9 warnings
the borrowed expression implements the required traits:
src/templates/filters.rs#L26
warning: the borrowed expression implements the required traits
--> src/templates/filters.rs:26:21
|
26 | Ok(to_value(&html).unwrap())
| ^^^^^ help: change this to: `html`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/rss.rs#L56
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/rss.rs:56:62
|
56 | let html_content = convert_md_to_html(&post.content, &settings, Some(&post.path[..]));
| ^^^^^^^^^ help: change this to: `settings`
|
= 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
|
the borrowed expression implements the required traits:
src/content.rs#L255
warning: the borrowed expression implements the required traits
--> src/content.rs:255:30
|
255 | match fs::read_to_string(&file_path) {
| ^^^^^^^^^^ help: change this to: `file_path`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
src/content.rs#L239
warning: the borrowed expression implements the required traits
--> src/content.rs:239:24
|
239 | fs::create_dir_all(&output_directory)
| ^^^^^^^^^^^^^^^^^ help: change this to: `output_directory`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
= note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default
|
use of `or_insert_with` to construct default value:
src/content.rs#L200
warning: use of `or_insert_with` to construct default value
--> src/content.rs:200:62
|
200 | let posts = indexes_to_create.entry(key).or_insert_with(Vec::new);
| ^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `or_default()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_or_default
= note: `#[warn(clippy::unwrap_or_default)]` on by default
|
this expression can be written more simply using `.retain()`:
src/content.rs#L105
warning: this expression can be written more simply using `.retain()`
--> src/content.rs:105:5
|
105 | / events = events
106 | | .into_iter()
107 | | .filter(|e| match e {
108 | | Event::Text(text) | Event::Html(text) => !text.is_empty(),
109 | | _ => true,
110 | | })
111 | | .collect();
| |__________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_retain
= note: `#[warn(clippy::manual_retain)]` on by default
help: consider calling `.retain()` instead
|
105 ~ events.retain(|e| match e {
106 + Event::Text(text) | Event::Html(text) => !text.is_empty(),
107 + _ => true,
108 ~ });
|
|
unneeded late initialization:
src/content.rs#L78
warning: unneeded late initialization
--> src/content.rs:78:21
|
78 | let html;
| ^^^^^^^^^ created here
79 | html = code_block.highlight(&text);
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ initialised here
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_late_init
= note: `#[warn(clippy::needless_late_init)]` on by default
help: declare `html` here
|
79 | let html = code_block.highlight(&text);
| ~~~~~~~~
|
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/
|