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

Pass CI #79

Merged
merged 4 commits into from
Jul 10, 2024
Merged

Pass CI #79

merged 4 commits into from
Jul 10, 2024

Conversation

SergioRibera
Copy link
Contributor

I made some improvements apart from what clippy recommends, I was removing the &String by &str and &Vec<_> by &[_], this because of these rules

I also fixed some things so that the tests can pass without problems, the code examples in the documentation comments are executed in the tests and we had one that has dependencies outside of our Cargo.toml, so I made Rust ignore that test to avoid problems.

this close #77

@SergioRibera SergioRibera added the ci This issue is related to CI label Jul 9, 2024
@SergioRibera SergioRibera added this to the Road to 0.2.0 milestone Jul 9, 2024
@SergioRibera SergioRibera requested a review from danmt July 9, 2024 18:30
@SergioRibera SergioRibera self-assigned this Jul 9, 2024
Copy link
Contributor

@danmt danmt left a comment

Choose a reason for hiding this comment

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

LGTM

@danmt danmt merged commit 8201c6c into master Jul 10, 2024
6 checks passed
@danmt danmt deleted the fix_clippy branch July 10, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This issue is related to CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add clippy suggestions to increase code quality
2 participants