rustype/typestate-rs

Generated Charts Thoughts

Closed this issue · 4 comments

Michael-J-Ward commented

I think the auto-gen state charts guaranteed to stay in line with the code is a killer feature.

A few bike-shedding UX improvements I wanted to see what you think

  • Allow ChoiceNode edge labels via annotation to the ChoiceEnum variants - either through an annotation #[label("out of retries")] or by grabbing the variant's doc comment like the displaydoc crate

  • Configurable asset destination - instead of populating the root directory, allow the user to configure it like by setting an env variable.

Are you accepting outside contributions?

jmg-duarte commented

Regarding your suggestions

  1. That is a great idea but the underlying graph would require some refactoring. We can discuss these changes as your suggestion makes a lot of sense and could be expanded in some cases.
  2. That is easy enough to do, I'll address that ASAP.

Regarding outside contributions, I'll need to get back to you on that, since the crate is part of my thesis project I need to check with my advisor.

  • 👍1
jmg-duarte commented

The latest version 0.7.1 addresses the destination issue

  • 👍1
jmg-duarte commented

@Michael-J-Ward I've checked and outside contributions are being accepted!