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.

Sign up for GitHub

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

Jump to bottom

run cargo fmt + cargo clippy + update GA-CI #31

Merged
merged 6 commits into from
Aug 10, 2021
Merged

run cargo fmt + cargo clippy + update GA-CI #31

merged 6 commits into from
Aug 10, 2021

Conversation

clangenb
Copy link
Contributor

@clangenb clangenb commented Aug 10, 2021 •
edited
Loading

GitHub Actions:

  • Add cargo fmt
  • Add cargo clippy

Code:

  • run cargo fmt on the whole codebase
  • fixed clippy warnings were it made sense

Other:

  • moved from legacy rust-toolchain file to richer rust-toolchain.toml

  • Closes Update CI #30

@clangenb
Copy link
Contributor Author

clangenb commented Aug 10, 2021 •
edited
Loading

@mosonyi FYI

@clangenb clangenb changed the title Cl update ci run cargo fmt + cargo clippy + update GA-CI Aug 10, 2021
@brenzi brenzi merged commit e1084fa into master Aug 10, 2021
@clangenb clangenb deleted the cl-update-ci branch August 19, 2021 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brenzi brenzi Awaiting requested review from brenzi

@mosonyi mosonyi Awaiting requested review from mosonyi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update CI
2 participants
@clangenb @brenzi