search

korken89 / smlang-rs

A State Machine Language DSL procedual macro for Rust
Apache License 2.0
203 stars 28 forks source link

Fix clippy warnings #59

Closed oblique closed 2 years ago

oblique commented 2 years ago

This fixes clippy warnings and adds clippy in CI. I also added a missing test case.

There is a small caveat by having clippy in CI: When someone opens a PR, their CI pipeline may fail in places they never touched. This is because newer versions of clippy have for more lints.

danieleades commented 2 years ago

There is a small caveat by having clippy in CI: When someone opens a PR, their CI pipeline may fail in places they never touched. This is because newer versions of clippy have for more lints.

you can address this by pinning the compiler version for that job in CI