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

Remove message module #8

Merged
merged 1 commit into from
Aug 6, 2017
Merged

Remove message module #8

merged 1 commit into from
Aug 6, 2017

Conversation

andrejewski
Copy link
Owner

@andrejewski andrejewski commented Aug 6, 2017

This removes the raj/message module.

Presenting Raj to a group it seemed that the focus fell onto the syntax of pattern matching and how state changes are defined. Those concerns are not really relevant to Raj as there is no assumption made how you model your state changes, just that there are state changes.

While I prefer raj/message style messages and pattern matching, I think it is going to be problematic for people trying to build around Raj if raj/message is taken to be the approved way of doing things, especially since the API would have been very minimal forever.

The module had only ever been included for convenience and had no strong ties to any other module. Before it becomes too pivotal, we should remove it.

I have created tagmeme which takes raj/message and improves upon my original API and is even more strict about the style I prefer. This means someone can create a library that better suits their use case as well. It is no longer a Raj concern.

@andrejewski andrejewski merged commit 1ea7654 into master Aug 6, 2017
@andrejewski andrejewski deleted the no-message branch August 6, 2017 19:00
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

Successfully merging this pull request may close these issues.

1 participant