add linting CI jobs #1
Annotations
9 warnings
field `icon` is never read:
cosmic-panel-button/src/main.rs#L9
warning: field `icon` is never read
--> cosmic-panel-button/src/main.rs:9:5
|
7 | struct Desktop {
| ------- field in this struct
8 | name: String,
9 | icon: Option<String>,
| ^^^^
|
= note: `Desktop` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
= note: `#[warn(dead_code)]` on by default
|
variant `Exit` is never constructed:
cosmic-app-list/src/wayland_subscription.rs#L100
warning: variant `Exit` is never constructed
--> cosmic-app-list/src/wayland_subscription.rs:100:5
|
97 | pub enum ToplevelRequest {
| --------------- variant in this enum
...
100 | Exit,
| ^^^^
|
= note: `ToplevelRequest` 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
|
variants `SetPairable` and `SetDiscoverable` are never constructed:
cosmic-applet-bluetooth/src/bluetooth.rs#L121
warning: variants `SetPairable` and `SetDiscoverable` are never constructed
--> cosmic-applet-bluetooth/src/bluetooth.rs:121:5
|
119 | pub enum BluerRequest {
| ------------ variants in this enum
120 | SetBluetoothEnabled(bool),
121 | SetPairable(bool),
| ^^^^^^^^^^^
122 | SetDiscoverable(bool),
| ^^^^^^^^^^^^^^^
|
= note: `BluerRequest` 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
|
non-canonical implementation of `partial_cmp` on an `Ord` type:
cosmic-applet-bluetooth/src/bluetooth.rs#L188
warning: non-canonical implementation of `partial_cmp` on an `Ord` type
--> cosmic-applet-bluetooth/src/bluetooth.rs:188:1
|
188 | / impl PartialOrd for BluerDevice {
189 | | fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
| | _______________________________________________________________________-
190 | || match self.status.cmp(&other.status) {
191 | || std::cmp::Ordering::Equal => {
192 | || Some(self.name.to_lowercase().cmp(&other.name.to_lowercase()))
... ||
195 | || }
196 | || }
| ||_____- help: change this to: `{ Some(self.cmp(other)) }`
197 | | }
| |__^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl
= note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default
|
variant `Finished` is never constructed:
cosmic-applet-audio/src/mpris_subscription.rs#L89
warning: variant `Finished` is never constructed
--> cosmic-applet-audio/src/mpris_subscription.rs:89:5
|
86 | pub enum MprisUpdate {
| ----------- variant in this enum
...
89 | Finished,
| ^^^^^^^^
|
= note: `MprisUpdate` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
= note: `#[warn(dead_code)]` on by default
|
non-binding `let` on a future:
cosmic-applet-audio/src/main.rs#L447
warning: non-binding `let` on a future
--> cosmic-applet-audio/src/main.rs:447:25
|
447 | / _ = tokio::spawn(async move {
448 | | tokio::time::sleep(tokio::time::Duration::from_secs(30)).await;
449 | | conn.send(pulse::Message::UpdateConnection);
450 | | });
| |__________________________^
|
= help: consider awaiting the future or dropping explicitly with `std::mem::drop`
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_future
= note: `#[warn(clippy::let_underscore_future)]` on by default
|
this function has too many arguments (9/7):
cosmic-applet-notifications/src/subscriptions/freedesktop_proxy.rs#L24
warning: this function has too many arguments (9/7)
--> cosmic-applet-notifications/src/subscriptions/freedesktop_proxy.rs:24:1
|
24 | // #[dbus_proxy(
25 | || interface = "org.freedesktop.Notifications",
26 | || default_service = "org.freedesktop.Notifications",
27 | || default_path = "/org/freedesktop/Notifications"
28 | || )]
| ||__- in this procedural macro expansion
... |
49 | | expire_timeout: i32,
50 | | ) -> zbus::Result<u32>;
| |___________________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
= note: `#[warn(clippy::too_many_arguments)]` on by default
= note: this warning originates in the attribute macro `dbus_proxy` (in Nightly builds, run with -Z macro-backtrace for more info)
|
this function has too many arguments (9/7):
cosmic-applet-notifications/src/subscriptions/freedesktop_proxy.rs#L24
warning: this function has too many arguments (9/7)
--> cosmic-applet-notifications/src/subscriptions/freedesktop_proxy.rs:24:1
|
24 | / #[dbus_proxy(
25 | | interface = "org.freedesktop.Notifications",
26 | | default_service = "org.freedesktop.Notifications",
27 | | default_path = "/org/freedesktop/Notifications"
28 | | )]
| |__^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
= note: this warning originates in the attribute macro `dbus_proxy` (in Nightly builds, run with -Z macro-backtrace for more info)
|
variant `Errored` is never constructed:
cosmic-applet-workspaces/src/components/app.rs#L40
warning: variant `Errored` is never constructed
--> cosmic-applet-workspaces/src/components/app.rs:40:5
|
36 | enum Message {
| ------- variant in this enum
...
40 | Errored,
| ^^^^^^^
|
= note: `Message` 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
|