test invalid Specta attribute formats #245
Annotations
8 errors and 8 warnings
build (ubuntu-latest)
ENOENT: no such file or directory, opendir '/home/runner/work/specta/specta/target/tests/target'
|
build (ubuntu-latest)
Error: ENOENT: no such file or directory, opendir '/home/runner/work/specta/specta/target/tests/target'
|
build (ubuntu-latest)
ENOENT: no such file or directory, opendir '/home/runner/work/specta/specta/target/tests/target'
|
build (ubuntu-latest)
Error: ENOENT: no such file or directory, opendir '/home/runner/work/specta/specta/target/tests/target'
|
build (windows-latest)
ENOENT: no such file or directory, opendir 'D:\a\specta\specta\target\tests\target'
|
build (windows-latest)
Error: ENOENT: no such file or directory, opendir 'D:\a\specta\specta\target\tests\target'
|
build (windows-latest)
ENOENT: no such file or directory, opendir 'D:\a\specta\specta\target\tests\target'
|
build (windows-latest)
Error: ENOENT: no such file or directory, opendir 'D:\a\specta\specta\target\tests\target'
|
length comparison to zero:
src/serde.rs#L164
warning: length comparison to zero
--> src/serde.rs:164:31
|
164 | if valid_variants == 0 && e.variants().len() != 0 {
| ^^^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!e.variants().is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
|
useless conversion to the same type: `std::string::String`:
src/lang/js_doc.rs#L44
warning: useless conversion to the same type: `std::string::String`
--> src/lang/js_doc.rs:44:21
|
44 | .chain([format!(r#"@typedef {{ {inline_ts} }} {name}"#).into()]),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing `.into()`: `format!(r#"@typedef {{ {inline_ts} }} {name}"#)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
= note: `#[warn(clippy::useless_conversion)]` implied by `#[warn(clippy::all)]`
|
called `.nth(0)` on a `std::iter::Iterator`, when `.next()` is equivalent:
src/lang/ts/mod.rs#L629
warning: called `.nth(0)` on a `std::iter::Iterator`, when `.next()` is equivalent
--> src/lang/ts/mod.rs:629:31
|
629 | if let Some(first_char) = ident.chars().nth(0) {
| ^^^^^^^^^^^^^^^^^^^^ help: try calling `.next()` instead of `.nth(0)`: `ident.chars().next()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#iter_nth_zero
= note: `#[warn(clippy::iter_nth_zero)]` implied by `#[warn(clippy::all)]`
|
length comparison to zero:
src/lang/ts/mod.rs#L427
warning: length comparison to zero
--> src/lang/ts/mod.rs:427:24
|
427 | if obj.fields.len() == 0 {
| ^^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `obj.fields.is_empty()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
= note: `#[warn(clippy::len_zero)]` implied by `#[warn(clippy::all)]`
|
using a reference to `Cow` is not recommended:
src/lang/ts/mod.rs#L103
warning: using a reference to `Cow` is not recommended
--> src/lang/ts/mod.rs:103:11
|
103 | docs: &Cow<'static, str>,
| ^^^^^^^^^^^^^^^^^^ help: change this to: `&str`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
= note: `#[warn(clippy::ptr_arg)]` implied by `#[warn(clippy::all)]`
|
calling `push_str()` using a single-character string literal:
src/lang/ts/comments.rs#L40
warning: calling `push_str()` using a single-character string literal
--> src/lang/ts/comments.rs:40:13
|
40 | comment.push_str(" ");
| ^^^^^^^^^^^^^^^^^^^^^ help: consider using `push` with a character literal: `comment.push(' ')`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_add_str
note: the lint level is defined here
--> src/lib.rs:3:9
|
3 | #![warn(clippy::all, clippy::unwrap_used, clippy::panic)] // TODO: missing_docs
| ^^^^^^^^^^^
= note: `#[warn(clippy::single_char_add_str)]` implied by `#[warn(clippy::all)]`
|
constant `RESERVED_IDENTS` is never used:
src/lang/ts/reserved_terms.rs#L66
warning: constant `RESERVED_IDENTS` is never used
--> src/lang/ts/reserved_terms.rs:66:18
|
66 | pub(crate) const RESERVED_IDENTS: &[&str] = &[
| ^^^^^^^^^^^^^^^
|
= note: `#[warn(dead_code)]` 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/
|