Skip to content

Add bevy

Add bevy #29

Triggered via pull request March 18, 2024 22:42
@berkusberkus
synchronize #5
add-bevy
Status Success
Total duration 8m 0s
Artifacts

codecov.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

14 warnings
use of `expect` followed by a function call: src/bin/convert_to_png.rs#L52
warning: use of `expect` followed by a function call --> src/bin/convert_to_png.rs:52:14 | 52 | .expect(format!("Couldnt create png file {:?}", pngname).as_ref()); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `unwrap_or_else(|_| panic!("Couldnt create png file {:?}", pngname))` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#expect_fun_call
use of `expect` followed by a function call: src/bin/convert_to_png.rs#L41
warning: use of `expect` followed by a function call --> src/bin/convert_to_png.rs:41:10 | 41 | .expect(format!("Couldnt open pix file {:?}", fname).as_ref()); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `unwrap_or_else(|_| panic!("Couldnt open pix file {:?}", fname))` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#expect_fun_call = note: `#[warn(clippy::expect_fun_call)]` on by default
field assignment outside of initializer for an instance created with Default::default(): src/support/brender/model.rs#L141
warning: field assignment outside of initializer for an instance created with Default::default() --> src/support/brender/model.rs:141:21 | 141 | model.name = model_chunk.identifier; | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | note: consider initializing the variable with `support::brender::model::Model { name: model_chunk.identifier, ..Default::default() }` and removing relevant reassignments --> src/support/brender/model.rs:140:21 | 140 | let mut model = Model::default(); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#field_reassign_with_default = note: `#[warn(clippy::field_reassign_with_default)]` on by default
called `map(..).flatten()` on `Iterator`: src/support/brender/model.rs#L57
warning: called `map(..).flatten()` on `Iterator` --> src/support/brender/model.rs:57:18 | 57 | .map(|f| [f.v1, f.v2, f.v3]) | __________________^ 58 | | .flatten() | |__________________________^ help: try replacing `map` with `flat_map` and remove the `.flatten()`: `flat_map(|f| [f.v1, f.v2, f.v3])` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#map_flatten = note: `#[warn(clippy::map_flatten)]` on by default
`Box::new(_)` of default value: src/support/brender/actor.rs#L106
warning: `Box::new(_)` of default value --> src/support/brender/actor.rs:106:35 | 106 | actor.model = Box::new(Model::default()); //Models::find(model.identifier); // World::query<Model>? | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `Box::<Model>::default()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#box_default = note: `#[warn(clippy::box_default)]` on by default
returning the result of a `let` binding from a block: src/support/brender/actor.rs#L190
warning: returning the result of a `let` binding from a block --> src/support/brender/actor.rs:190:9 | 189 | let actor = stack.pop::<Actor>()?; | ---------------------------------- unnecessary `let` binding 190 | actor | ^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return = note: `#[warn(clippy::let_and_return)]` on by default help: return the expression directly | 189 ~ 190 ~ stack.pop::<Actor>()? |
`Vec<T>` is already on the heap, the boxing is unnecessary: src/support/brender/actor.rs#L66
warning: `Vec<T>` is already on the heap, the boxing is unnecessary --> src/support/brender/actor.rs:66:15 | 66 | children: Vec<Box<Actor>>, | ^^^^^^^^^^^^^^^ help: try: `Vec<Actor>` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#vec_box = note: `#[warn(clippy::vec_box)]` on by default
Test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions-rs/toolchain@v1, actions/cache@v2, codecov/codecov-action@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1, actions/cache@v2, codecov/codecov-action@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Test
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/