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.

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

question: enumerable event properties #6

Open
boneskull opened this issue Oct 18, 2015 · 3 comments
Open

question: enumerable event properties #6

boneskull opened this issue Oct 18, 2015 · 3 comments

Comments

@boneskull
Copy link
Contributor

I'm curious why "event" properties are enumerable when other properties (such as can and cannot) are not?

@vstirbu
Copy link
Owner

vstirbu commented Oct 18, 2015

The idea at the time was that the public api surface of the fem object is just the events set, the rest would be private. The private api was used for testing or just compatibility with the inspiration.

@boneskull
Copy link
Contributor Author

is it still necessary to have them either way?

@vstirbu
Copy link
Owner

vstirbu commented Oct 21, 2015

They can be made enumerable and in addition the lib should throw an error if the state machine definition contains any event names that would conflict with the library added methods: is, can, cannot and current.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants