Skip to content
GitHub Actions / clippy succeeded Feb 3, 2025 in 0s

clippy

17 warnings

Details

Results

Message level Amount
Internal compiler error 0
Error 0
Warning 17
Note 0
Help 0

Versions

  • rustc 1.84.1 (e71f9a9a9 2025-01-27)
  • cargo 1.84.1 (66221abde 2024-11-19)
  • clippy 0.1.84 (e71f9a9a98 2025-01-27)

Annotations

Check warning on line 23 in partiql-conformance-tests/src/bin/generate_comparison_report.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

doc list item without indentation

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)
   |     ++

Check warning on line 353 in partiql-logical-planner/src/typer.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unreachable pattern

warning: unreachable pattern
   --> partiql-logical-planner/src/typer.rs:353:21
    |
353 |                     _ => {
    |                     ^ no value can reach this
    |
note: multiple earlier patterns match some of the same values
   --> partiql-logical-planner/src/typer.rs:353:21
    |
341 |                     Lit::Null | Lit::Missing => self.bld.new_undefined(),
    |                     ------------------------ matches some of the same values
342 |                     Lit::Int8(_) | Lit::Int16(_) | Lit::Int32(_) | Lit::Int64(_) => {
    |                     ------------------------------------------------------------ matches some of the same values
...
345 |                     Lit::Decimal(_) => type_decimal!(self.bld),
    |                     --------------- matches some of the same values
346 |                     Lit::Double(_) => type_float64!(self.bld),
    |                     -------------- matches some of the same values
...
353 |                     _ => {
    |                     ^ ...and 6 other patterns collectively make this unreachable
    = note: `#[warn(unreachable_patterns)]` on by default

Check warning on line 53 in partiql-eval/src/eval/eval_expr_wrapper.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `lower`

warning: unused variable: `lower`
  --> partiql-eval/src/eval/eval_expr_wrapper.rs:53:38
   |
53 |                 DatumValueRef::Lower(lower) => {
   |                                      ^^^^^ help: if this is intentional, prefix it with an underscore: `_lower`
   |
   = note: `#[warn(unused_variables)]` on by default

Check warning on line 11 in extension/partiql-extension-ion/src/util.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

variant `Bag` is never constructed

warning: variant `Bag` is never constructed
  --> extension/partiql-extension-ion/src/util.rs:11:5
   |
8  | pub enum PartiqlValueTarget<T> {
   |          ------------------ variant in this enum
...
11 |     Bag(Vec<T>),
   |     ^^^
   |
   = note: `#[warn(dead_code)]` on by default

Check warning on line 379 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

type `boxed_ion::BoxedIonStreamType` is more private than the item `boxed_ion::BoxedIon::parse`

warning: type `boxed_ion::BoxedIonStreamType` is more private than the item `boxed_ion::BoxedIon::parse`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:379:5
    |
379 |     pub fn parse(data: Vec<u8>, expected: BoxedIonStreamType) -> IonResult<Self> {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ associated function `boxed_ion::BoxedIon::parse` is reachable at visibility `pub`
    |
note: but type `boxed_ion::BoxedIonStreamType` is only usable at visibility `pub(self)`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:451:1
    |
451 | enum BoxedIonStreamType {
    | ^^^^^^^^^^^^^^^^^^^^^^^

Check warning on line 364 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

type `boxed_ion::IonContext` is more private than the item `boxed_ion::BoxedIon::new_value`

warning: type `boxed_ion::IonContext` is more private than the item `boxed_ion::BoxedIon::new_value`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:364:5
    |
364 |     pub fn new_value(doc: impl Into<BoxedIonValue>, ctx: IonContextPtr) -> Value {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ associated function `boxed_ion::BoxedIon::new_value` is reachable at visibility `pub`
    |
note: but type `boxed_ion::IonContext` is only usable at visibility `pub(self)`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:66:1
    |
66  | struct IonContext {
    | ^^^^^^^^^^^^^^^^^

Check warning on line 364 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

type `boxed_ion::BoxedIonValue` is more private than the item `boxed_ion::BoxedIon::new_value`

warning: type `boxed_ion::BoxedIonValue` is more private than the item `boxed_ion::BoxedIon::new_value`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:364:5
    |
364 |     pub fn new_value(doc: impl Into<BoxedIonValue>, ctx: IonContextPtr) -> Value {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ associated function `boxed_ion::BoxedIon::new_value` is reachable at visibility `pub`
    |
note: but type `boxed_ion::BoxedIonValue` is only usable at visibility `pub(self)`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:458:1
    |
458 | enum BoxedIonValue {
    | ^^^^^^^^^^^^^^^^^^

Check warning on line 358 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

type `boxed_ion::IonContext` is more private than the item `boxed_ion::BoxedIon::new`

warning: type `boxed_ion::IonContext` is more private than the item `boxed_ion::BoxedIon::new`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:358:5
    |
358 |     pub fn new(doc: impl Into<BoxedIonValue>, ctx: IonContextPtr) -> Self {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ associated function `boxed_ion::BoxedIon::new` is reachable at visibility `pub`
    |
note: but type `boxed_ion::IonContext` is only usable at visibility `pub(self)`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:66:1
    |
66  | struct IonContext {
    | ^^^^^^^^^^^^^^^^^

Check warning on line 358 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

type `boxed_ion::BoxedIonValue` is more private than the item `boxed_ion::BoxedIon::new`

warning: type `boxed_ion::BoxedIonValue` is more private than the item `boxed_ion::BoxedIon::new`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:358:5
    |
358 |     pub fn new(doc: impl Into<BoxedIonValue>, ctx: IonContextPtr) -> Self {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ associated function `boxed_ion::BoxedIon::new` is reachable at visibility `pub`
    |
note: but type `boxed_ion::BoxedIonValue` is only usable at visibility `pub(self)`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:458:1
    |
458 | enum BoxedIonValue {
    | ^^^^^^^^^^^^^^^^^^
    = note: `#[warn(private_bounds)]` on by default

Check warning on line 82 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

type `boxed_ion::IonContext` is more private than the item `boxed_ion::IonContextPtr`

warning: type `boxed_ion::IonContext` is more private than the item `boxed_ion::IonContextPtr`
  --> extension/partiql-extension-ion/src/boxed_ion.rs:82:1
   |
82 | pub type IonContextPtr = Rc<RefCell<IonContext>>;
   | ^^^^^^^^^^^^^^^^^^^^^^ type alias `boxed_ion::IonContextPtr` is reachable at visibility `pub`
   |
note: but type `boxed_ion::IonContext` is only usable at visibility `pub(self)`
  --> extension/partiql-extension-ion/src/boxed_ion.rs:66:1
   |
66 | struct IonContext {
   | ^^^^^^^^^^^^^^^^^
   = note: `#[warn(private_interfaces)]` on by default

Check warning on line 310 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `k`

warning: unused variable: `k`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:310:23
    |
310 |     fn take_val(self, k: &BindingsName<'_>) -> Option<Value> {
    |                       ^ help: if this is intentional, prefix it with an underscore: `_k`

Check warning on line 103 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `deserializer`

warning: unused variable: `deserializer`
   --> extension/partiql-extension-ion/src/boxed_ion.rs:103:23
    |
103 |     fn deserialize<D>(deserializer: D) -> std::result::Result<Self, D::Error>
    |                       ^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_deserializer`

Check warning on line 93 in extension/partiql-extension-ion/src/boxed_ion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused variable: `serializer`

warning: unused variable: `serializer`
  --> extension/partiql-extension-ion/src/boxed_ion.rs:93:28
   |
93 |     fn serialize<S>(&self, serializer: S) -> std::result::Result<S::Ok, S::Error>
   |                            ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_serializer`
   |
   = note: `#[warn(unused_variables)]` on by default

Check warning on line 5 in extension/partiql-extension-ion/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused imports: `Deserialize` and `Serialize`

warning: unused imports: `Deserialize` and `Serialize`
 --> extension/partiql-extension-ion/src/lib.rs:5:13
  |
5 | use serde::{Deserialize, Serialize};
  |             ^^^^^^^^^^^  ^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

Check warning on line 115 in partiql-value/src/variant.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

field `0` is never read

warning: field `0` is never read
   --> partiql-value/src/variant.rs:115:28
    |
115 | pub struct VariantIter<'a>(BoxedVariantValueIter<'a>);
    |            -----------     ^^^^^^^^^^^^^^^^^^^^^^^^^
    |            |
    |            field in this struct
    |
    = help: consider removing this field
    = note: `#[warn(dead_code)]` on by default

Check warning on line 36 in partiql-value/src/boxed_variant.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unnecessary associated type bound for dyn-incompatible associated type

warning: unnecessary associated type bound for dyn-incompatible associated type
  --> partiql-value/src/boxed_variant.rs:36:60
   |
36 | pub type BoxedVariantTypeTag<D> = Box<dyn BoxedVariantType<Doc = D>>;
   |                                                            ^^^^^^^ help: remove this bound
   |
   = note: this associated type has a `where Self: Sized` bound, and while the associated type can be specified, it cannot be used because trait objects are never `Sized`
   = note: `#[warn(unused_associated_type_bounds)]` on by default

Check warning on line 23 in partiql-value/src/lib.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unused import: `pretty::*`

warning: unused import: `pretty::*`
  --> partiql-value/src/lib.rs:23:9
   |
23 | pub use pretty::*;
   |         ^^^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default