Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch Style and updates version 0.5 #30

Merged
merged 6 commits into from
Jul 1, 2024
Merged

Conversation

naomijub
Copy link
Contributor

Hi @tasgon, I updated the Cargo.toml to version 0.5 and did some minor coding style changes that make your code more expressive.

Also:

  • Added const to a couple of functions that were const.
  • Added _else clause to some of your map/or because they were allocating memory and it was not necessary to do it eagerly.
  • Ran cargo update to update your Cargo.lock

examples/fonts.rs Show resolved Hide resolved
src/utils.rs Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@naomijub
Copy link
Contributor Author

@tasgon Do you want me to chance the comment you made or did my responses clarify?

@naomijub
Copy link
Contributor Author

@tasgon ?

@tasgon
Copy link
Owner

tasgon commented Jun 30, 2024

Sorry -- looks good. I'll double check it tomorrow, and if I don't see anything else I'll merge and publish.

@tasgon tasgon merged commit 7b9f496 into tasgon:master Jul 1, 2024
2 checks passed
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