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

Add Enterprise #172

Closed
wants to merge 1 commit into from
Closed

Add Enterprise #172

wants to merge 1 commit into from

Conversation

marcinzh
Copy link
Contributor

  1. Add Enterprise
  2. Fix formatting of my last submission: Turbolift (missing 2 spaces)

@yallop
Copy link
Owner

yallop commented Jul 29, 2023

I think Enterprise doesn't quite fit here. Libraries that implement effects and handlers (particularly if use notable implementation techniques, and particularly if they're used by others) are in scope, but libraries that simply use algebraic effects and handlers aren't.

@marcinzh
Copy link
Contributor Author

Well, I suppose I took "theory and practice" part of the decription too literally.

@yallop
Copy link
Owner

yallop commented Aug 10, 2023

Not too literally, exactly, just more broadly than we intended. Thank you for the PR; I've pulled out the formatting change into #177 to merge separately.

@yallop yallop closed this Aug 10, 2023
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.

2 participants