search

korken89 / smlang-rs

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

Add impl_display_{states,events} #64

Closed maximeborges closed 1 year ago

maximeborges commented 1 year ago

This PR adds generation of core::fmt::Display for Events and States by adding the corresponding impl_display_{states,events} field to the statemachine! declaration.

ryan-summers commented 1 year ago

Also, please add a note of these changes in CHANGELOG.md

ryan-summers commented 1 year ago

Mind fixing clippy?

maximeborges commented 1 year ago

Mind fixing clippy?

Somehow I don't get those errors locally, even with the same command ^^'

ryan-summers commented 1 year ago

Somehow I don't get those errors locally, even with the same command ^^'

I see this a lot myself and it's generally because by local version of clippy is older than what CI uses. This lint may have been recently release in rust 1.70