Skip to content

Chore/update project - #85

Open
MarioYellowy wants to merge 17 commits into
RustLangES:riirfrom
MarioYellowy:chore/update-project
Open

Chore/update project#85
MarioYellowy wants to merge 17 commits into
RustLangES:riirfrom
MarioYellowy:chore/update-project

Conversation

@MarioYellowy

@MarioYellowy MarioYellowy commented Aug 5, 2026

Copy link
Copy Markdown

Changes implemented:
On flake.nix was updated the library for use:

  • wasm-bindgen-cli_0_2_126
  • pnpm

Update the version of rust:

  • nightly-2026-08-05

Update many dependencies in cargo.toml

Reconfigure the README file for indicate a better way to configure the project throw make commands.

Comment on lines -5 to -12
#[prop(default = 40)] size: u32,
#[prop(default = "dark:fill-[#e2cea9]")] class: &'static str,
#[prop(default = 40)] _size: u32,
#[prop(default = "dark:fill-[#e2cea9]")] _class: &'static str,
) -> impl IntoView {
view! {
<svg
width=size
height=size
class=class

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why the _?

@MarioYellowy MarioYellowy Aug 5, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently I didn't made a refactorization in the most of the code, there is a plan behind of this, currently I focus only in the warning/errors of clippy, clippy mark as unused, that's why
pdt: Assimen me to this pull request, greetings

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's used below, we might need to find another way to prevent Clippy from popping up, but the _ usually makes the code look messy if it's actually being used

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The refactorization are comming, the flake se pudrió

@MarioYellowy
MarioYellowy marked this pull request as ready for review August 7, 2026 00:56
@gg0074x
gg0074x requested a review from Brayan-724 August 7, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants