cleanup #63
BrendonovichAnnotations
5 errors and 20 warnings
|
build (macos-latest)
Process completed with exit code 101.
|
|
build (windows-latest)
The job was canceled because "macos-latest" failed.
|
|
build (windows-latest)
The operation was canceled.
|
|
build (ubuntu-latest)
The job was canceled because "macos-latest" failed.
|
|
build (ubuntu-latest)
The operation was canceled.
|
|
used `unwrap()` on a `Result` value:
src/lib.rs#L362
warning: used `unwrap()` on a `Result` value
--> src/lib.rs:362:50
|
362 | let (invoke_handler, event_collection) = self.export_inner().unwrap();
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: if you don't want to handle the `Err` case gracefully, consider using `expect()` to provide a better panic message
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
note: the lint level is defined here
--> src/lib.rs:91:22
|
91 | #![warn(clippy::all, clippy::unwrap_used, clippy::panic
| ^^^^^^^^^^^^^^^^^^^
|
|
the borrowed expression implements the required traits:
src/event.rs#L189
warning: the borrowed expression implements the required traits
--> src/event.rs:189:21
|
189 | window.once(&meta.wrap_with_plugin(Self::NAME), make_handler!(handler))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `meta.wrap_with_plugin(Self::NAME)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
the borrowed expression implements the required traits:
src/event.rs#L180
warning: the borrowed expression implements the required traits
--> src/event.rs:180:23
|
180 | window.listen(&meta.wrap_with_plugin(Self::NAME), make_handler!(handler))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `meta.wrap_with_plugin(Self::NAME)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
the borrowed expression implements the required traits:
src/event.rs#L148
warning: the borrowed expression implements the required traits
--> src/event.rs:148:28
|
148 | handle.once_global(&meta.wrap_with_plugin(Self::NAME), make_handler!(handler))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `meta.wrap_with_plugin(Self::NAME)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
the borrowed expression implements the required traits:
src/event.rs#L139
warning: the borrowed expression implements the required traits
--> src/event.rs:139:30
|
139 | handle.listen_global(&meta.wrap_with_plugin(Self::NAME), make_handler!(handler))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `meta.wrap_with_plugin(Self::NAME)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
|
|
`panic` should not be present in production code:
src/event.rs#L81
warning: `panic` should not be present in production code
--> src/event.rs:81:28
|
81 | .unwrap_or_else(|| panic!("Event {name} not found in registry!"))
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#panic
|
|
the following explicit lifetimes could be elided: 'a:
src/event.rs#L54
warning: the following explicit lifetimes could be elided: 'a
--> src/event.rs:54:26
|
54 | pub fn get_or_manage<'a, R: Runtime>(handle: &'a impl Manager<R>) -> tauri::State<'a, Self> {
| ^^ ^^ ^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
= note: `#[warn(clippy::needless_lifetimes)]` implied by `#[warn(clippy::all)]`
help: elide the lifetimes
|
54 - pub fn get_or_manage<'a, R: Runtime>(handle: &'a impl Manager<R>) -> tauri::State<'a, Self> {
54 + pub fn get_or_manage<R: Runtime>(handle: &impl Manager<R>) -> tauri::State<'_, Self> {
|
|
|
`panic` should not be present in production code:
src/event.rs#L34
warning: `panic` should not be present in production code
--> src/event.rs:34:13
|
34 | panic!("Another event with name {} is already registered!", E::NAME)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#panic
|
|
`panic` should not be present in production code:
src/event.rs#L30
warning: `panic` should not be present in production code
--> src/event.rs:30:13
|
30 | panic!("Event {} registered twice!", E::NAME)
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#panic
note: the lint level is defined here
--> src/lib.rs:91:43
|
91 | #![warn(clippy::all, clippy::unwrap_used, clippy::panic
| ^^^^^^^^^^^^^
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/js.rs#L190
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:190:51
|
190 | ts::named_datatype(&cfg.inner, v, &type_map).map(|typ| {
| ^^^^^^^^^ help: change this to: `type_map`
|
= 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/js.rs#L184
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:184:50
|
184 | let events = Self::render_events(events, &type_map, cfg)?;
| ^^^^^^^^^ help: change this to: `type_map`
|
= 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/js.rs#L183
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:183:56
|
183 | let commands = Self::render_commands(commands, &type_map, cfg)?;
| ^^^^^^^^^ help: change this to: `type_map`
|
= 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/js.rs#L136
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:136:38
|
136 | .apply_as_prefix(&event.name, ItemType::Event);
| ^^^^^^^^^^^ help: change this to: `event.name`
|
= 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/js.rs#L55
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:55:59
|
55 | ... ts::datatype(&cfg.inner, typ, &type_map).map(|typ| {
| ^^^^^^^^^ help: change this to: `type_map`
|
= 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/js.rs#L48
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:48:58
|
48 | t => ts::datatype(&cfg.inner, t, &type_map)?,
| ^^^^^^^^^ help: change this to: `type_map`
|
= 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/js.rs#L45
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:45:61
|
45 | ... ts::datatype(&cfg.inner, e, &type_map)?
| ^^^^^^^^^ help: change this to: `type_map`
|
= 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/js.rs#L44
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/js.rs:44:61
|
44 | ... ts::datatype(&cfg.inner, t, &type_map)?,
| ^^^^^^^^^ help: change this to: `type_map`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
= note: `#[warn(clippy::needless_borrow)]` implied by `#[warn(clippy::all)]`
|
|
this `.into_iter()` call is equivalent to `.iter()` and will not consume the `slice`:
src/js.rs#L35
warning: this `.into_iter()` call is equivalent to `.iter()` and will not consume the `slice`
--> src/js.rs:35:14
|
35 | .into_iter()
| ^^^^^^^^^ help: call directly: `iter`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#into_iter_on_ref
note: the lint level is defined here
--> src/lib.rs:91:9
|
91 | #![warn(clippy::all, clippy::unwrap_used, clippy::panic
| ^^^^^^^^^^^
= note: `#[warn(clippy::into_iter_on_ref)]` implied by `#[warn(clippy::all)]`
|
|
unused import: `borrow::Cow`:
src/lib.rs#L97
warning: unused import: `borrow::Cow`
--> src/lib.rs:97:5
|
97 | borrow::Cow,
| ^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
|
|
clippy
The following actions uses node12 which is deprecated and will be forced to run on node16: 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/
|