Skip to content

Cleanup & format

Cleanup & format #2825

Triggered via push November 18, 2024 19:51
Status Success
Total duration 7m 55s
Artifacts 1
Create conformance report for `push` and `pull_request` events
6m 7s
Create conformance report for `push` and `pull_request` events
Matrix: Build and Test
Matrix: cargo-deny
Create comparison report for `pull_request` event
0s
Create comparison report for `pull_request` event
Fit to window
Zoom out
Zoom in

Annotations

1 error and 54 warnings
Create conformance report for `push` and `pull_request` events
Process completed with exit code 101.
cargo-deny (bans licenses sources)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
cargo-deny (advisories)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
doc list item without indentation: partiql-conformance-tests/src/bin/generate_comparison_report.rs#L23
warning: doc list item without indentation --> partiql-conformance-tests/src/bin/generate_comparison_report.rs:23:5 | 23 | /// regressed behavior) | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation = note: `#[warn(clippy::doc_lazy_continuation)]` on by default help: indent this line | 23 | /// regressed behavior) | ++
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
unexpected `cfg` condition value: `serde`: partiql-eval/src/eval/mod.rs#L149
warning: unexpected `cfg` condition value: `serde` --> partiql-eval/src/eval/mod.rs:149:12 | 149 | #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] | ^^^^^^^^^^^^^^^^^ help: remove the condition | = note: no expected values for `feature` = help: consider adding `serde` as a feature in `Cargo.toml` = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration = note: `#[warn(unexpected_cfgs)]` on by default
variant `IllegalState` is never constructed: partiql-logical/src/util.rs#L80
warning: variant `IllegalState` is never constructed --> partiql-logical/src/util.rs:80:5 | 77 | pub enum LiteralError { | ------------ variant in this enum ... 80 | IllegalState(String), | ^^^^^^^^^^^^ | = note: `LiteralError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default
unused variable: `typ`: partiql-logical/src/util.rs#L61
warning: unused variable: `typ` --> partiql-logical/src/util.rs:61:40 | 61 | Lit::BoxDocument(contents, typ) => { | ^^^ help: if this is intentional, prefix it with an underscore: `_typ`
unused variable: `dt`: partiql-logical/src/util.rs#L20
warning: unused variable: `dt` --> partiql-logical/src/util.rs:20:29 | 20 | Value::DateTime(dt) => { | ^^ help: if this is intentional, prefix it with an underscore: `_dt`
unused variable: `bytes`: partiql-logical/src/util.rs#L17
warning: unused variable: `bytes` --> partiql-logical/src/util.rs:17:25 | 17 | Value::Blob(bytes) => { | ^^^^^ help: if this is intentional, prefix it with an underscore: `_bytes` | = note: `#[warn(unused_variables)]` on by default
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
unused imports: `Deserialize` and `Serialize`: partiql-value/src/lib.rs#L26
warning: unused imports: `Deserialize` and `Serialize` --> partiql-value/src/lib.rs:26:13 | 26 | use serde::{Deserialize, Serialize}; | ^^^^^^^^^^^ ^^^^^^^^^ | = note: `#[warn(unused_imports)]` on by default
doc list item without indentation: partiql-conformance-tests/src/bin/generate_comparison_report.rs#L23
warning: doc list item without indentation --> partiql-conformance-tests/src/bin/generate_comparison_report.rs:23:5 | 23 | /// regressed behavior) | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation = note: `#[warn(clippy::doc_lazy_continuation)]` on by default help: indent this line | 23 | /// regressed behavior) | ++
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
unexpected `cfg` condition value: `serde`: partiql-eval/src/eval/mod.rs#L149
warning: unexpected `cfg` condition value: `serde` --> partiql-eval/src/eval/mod.rs:149:12 | 149 | #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] | ^^^^^^^^^^^^^^^^^ help: remove the condition | = note: no expected values for `feature` = help: consider adding `serde` as a feature in `Cargo.toml` = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration = note: `#[warn(unexpected_cfgs)]` on by default
variant `IllegalState` is never constructed: partiql-logical/src/util.rs#L80
warning: variant `IllegalState` is never constructed --> partiql-logical/src/util.rs:80:5 | 77 | pub enum LiteralError { | ------------ variant in this enum ... 80 | IllegalState(String), | ^^^^^^^^^^^^ | = note: `LiteralError` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default
unused variable: `typ`: partiql-logical/src/util.rs#L61
warning: unused variable: `typ` --> partiql-logical/src/util.rs:61:40 | 61 | Lit::BoxDocument(contents, typ) => { | ^^^ help: if this is intentional, prefix it with an underscore: `_typ`
unused variable: `dt`: partiql-logical/src/util.rs#L20
warning: unused variable: `dt` --> partiql-logical/src/util.rs:20:29 | 20 | Value::DateTime(dt) => { | ^^ help: if this is intentional, prefix it with an underscore: `_dt`
unused variable: `bytes`: partiql-logical/src/util.rs#L17
warning: unused variable: `bytes` --> partiql-logical/src/util.rs:17:25 | 17 | Value::Blob(bytes) => { | ^^^^^ help: if this is intentional, prefix it with an underscore: `_bytes` | = note: `#[warn(unused_variables)]` on by default
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `Assertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
unused imports: `Deserialize` and `Serialize`: partiql-value/src/lib.rs#L26
warning: unused imports: `Deserialize` and `Serialize` --> partiql-value/src/lib.rs:26:13 | 26 | use serde::{Deserialize, Serialize}; | ^^^^^^^^^^^ ^^^^^^^^^ | = note: `#[warn(unused_imports)]` on by default
Build and Test (macos-latest)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Create conformance report for `push` and `pull_request` events
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/cache@v3, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Build and Test (ubuntu-20.04)
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/
Build and Test (ubuntu-20.04)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions-rs/clippy-check@v1, actions/cache@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Build and Test (windows-latest)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "artifact". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Artifacts

Produced during runtime
Name Size
artifact
1010 KB