Skip to content

Update the crates README, clean up dependencies #253

Update the crates README, clean up dependencies

Update the crates README, clean up dependencies #253

Triggered via push December 6, 2024 17:15
Status Success
Total duration 1m 4s
Artifacts
check_license
52s
check_license
Fit to window
Zoom out
Zoom in

Annotations

13 warnings
check_license
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
function `json_as_var` is never used: crates/web-host/src/host/mod.rs#L102
warning: function `json_as_var` is never used --> crates/web-host/src/host/mod.rs:102:8 | 102 | pub fn json_as_var(j: &serde_json::Value) -> Result<Var, JsonParseError> { | ^^^^^^^^^^^
variants `UnknownType`, `UnknownError`, and `InvalidRepresentation` are never constructed: crates/web-host/src/host/mod.rs#L95
warning: variants `UnknownType`, `UnknownError`, and `InvalidRepresentation` are never constructed --> crates/web-host/src/host/mod.rs:95:5 | 93 | pub enum JsonParseError { | -------------- variants in this enum 94 | #[error("Unknown type")] 95 | UnknownType, | ^^^^^^^^^^^ 96 | #[error("Unknown error")] 97 | UnknownError, | ^^^^^^^^^^^^ 98 | #[error("Invalid representation")] 99 | InvalidRepresentation, | ^^^^^^^^^^^^^^^^^^^^^ | = note: `JsonParseError` 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
very complex type used. Consider factoring parts into `type` definitions: crates/daemon/src/rpc_hosts.rs#L24
warning: very complex type used. Consider factoring parts into `type` definitions --> crates/daemon/src/rpc_hosts.rs:24:18 | 24 | pub struct Hosts(HashMap<HostToken, (SystemTime, HostType, Vec<(Obj, SocketAddr)>)>); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity = note: `#[warn(clippy::type_complexity)]` on by default
called `unwrap` on `path` after checking its variant with `is_none`: crates/daemon/src/connections_fjall.rs#L69
warning: called `unwrap` on `path` after checking its variant with `is_none` --> crates/daemon/src/connections_fjall.rs:69:20 | 64 | let (tmpdir, path) = if path.is_none() { | ----------------- help: try: `if let Some(<item>) = path` ... 69 | (None, path.unwrap().to_path_buf()) | ^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_unwrap = note: `#[warn(clippy::unnecessary_unwrap)]` on by default
very complex type used. Consider factoring parts into `type` definitions: crates/kernel/src/tasks/task.rs#L584
warning: very complex type used. Consider factoring parts into `type` definitions --> crates/kernel/src/tasks/task.rs:584:6 | 584 | ) -> Result<Option<((Bytes, VerbDef), Obj)>, CommandError> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#type_complexity = note: `#[warn(clippy::type_complexity)]` on by default
empty line after doc comment: crates/kernel/src/tasks/sessions.rs#L35
warning: empty line after doc comment --> crates/kernel/src/tasks/sessions.rs:35:1 | 35 | / /// 36 | | // TODO: Fix up connected/reconnected/discconnected handling. 37 | | // Will probably deprecate MOO's concept of 'disconnected' and 'connected' players in the long 38 | | // run and emulate slack, discord, skype, etc which have a concept of 'presence' (online, offline, ... | 42 | | // which is different from MOO's "reconnected" handling, but probably preferable. 43 | | | |_ 44 | pub trait Session: Send + Sync { | ------------------------------ the comment documents this trait | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments = note: `#[warn(clippy::empty_line_after_doc_comments)]` on by default = help: if the empty line is unintentional remove it help: if the doc comment should not document `Session` comment it out | 23 ~ // /// The interface for managing the user I/O connection side of state, exposed by the scheduler to 24 ~ // /// the VM during execution and by the host server to the scheduler. 25 ~ // /// 26 ~ // /// Because the execution path within the server is transactional, with the scheduler committing 27 ~ // /// and rolling back 'world state' on task commit/rollback, the general expectation is that this 28 ~ // /// entity should *also* perform transactionally, buffering output until the task commits, and 29 ~ // /// throwing it out on rollback. This may or may not be practical for large amounts of output. 30 ~ // /// 31 ~ // /// It is up to the implementation to decide how to buffer output. Options could include a 32 ~ // /// memory mapped file, a full database, or a simple in-memory buffer. 33 ~ // /// 34 ~ // /// Implementations would live in the 'server' host (e.g. websocket connections or repl loop) 35 ~ // /// |
variant name ends with the enum's name: crates/db/src/tx/mod.rs#L52
warning: variant name ends with the enum's name --> crates/db/src/tx/mod.rs:52:5 | 52 | EncodingError, | ^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names
variant name ends with the enum's name: crates/db/src/tx/mod.rs#L50
warning: variant name ends with the enum's name --> crates/db/src/tx/mod.rs:50:5 | 50 | StoreError, | ^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names
variant name ends with the enum's name: crates/db/src/tx/mod.rs#L48
warning: variant name ends with the enum's name --> crates/db/src/tx/mod.rs:48:5 | 48 | RetrieveError, | ^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#enum_variant_names = note: `#[warn(clippy::enum_variant_names)]` on by default
non-canonical implementation of `partial_cmp` on an `Ord` type: crates/db/src/tx/mod.rs#L24
warning: non-canonical implementation of `partial_cmp` on an `Ord` type --> crates/db/src/tx/mod.rs:24:1 | 24 | / impl PartialOrd for Timestamp { 25 | | fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> { | | _______________________________________________________________________- 26 | || self.0.partial_cmp(&other.0) 27 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 28 | | } | |__^ | = 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
empty line after doc comment: crates/db/src/worldstate_transaction.rs#L74
warning: empty line after doc comment --> crates/db/src/worldstate_transaction.rs:74:5 | 74 | / /// Get the parent of the given object. 75 | | | |_ 76 | fn get_object_parent(&self, obj: &Obj) -> Result<Obj, WorldStateError>; | ----------------------------------------------------------------------- the comment documents this method | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments = help: if the empty line is unintentional remove it
empty line after doc comment: crates/db/src/loader.rs#L31
warning: empty line after doc comment --> crates/db/src/loader.rs:31:1 | 31 | / /// WorldState could provide, but potentially different constraints/semantics (e.g. no perms checks) 32 | | | |_ 33 | pub trait LoaderInterface: Send { | ------------------------------- the comment documents this trait | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#empty_line_after_doc_comments = note: `#[warn(clippy::empty_line_after_doc_comments)]` on by default = help: if the empty line is unintentional remove it