-
Notifications
You must be signed in to change notification settings - Fork 760
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
Add ToDotGraph() method #33
Conversation
@@ -11,7 +11,7 @@ internal class StateRepresentation | |||
{ | |||
readonly TState _state; | |||
|
|||
readonly IDictionary<TTrigger, ICollection<TriggerBehaviour>> _triggerBehaviours = | |||
protected internal readonly IDictionary<TTrigger, ICollection<TriggerBehaviour>> _triggerBehaviours = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than making the fields protected internal
could we wrap them in internal
properties?
Looks great, just a couple of minor comments. Is it also possible to include a README update in the PR that notes this method and links to some DOT visualisation options? Just a thought :-) Cheers! |
Thanks for the feedback! Please let me know if there are any remaining/new issues or if you have more good suggestions. Also, I'll gladly squash the feature branch into a single commit if your prefer that. :-) |
Thanks, looks great. Looks like I need to get CI for this set up :-) |
Thanks! |
Proposal for nblumhardt/stateless/issues/31