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.

Sign up for GitHub

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

Jump to bottom

Seaside-Continuation package is missing #1007

Closed
jbrichau opened this issue Sep 10, 2018 · 1 comment
Closed

Seaside-Continuation package is missing #1007

jbrichau opened this issue Sep 10, 2018 · 1 comment
Milestone
3.3

Comments

@jbrichau
Copy link
Member

We have platform-specific packages for continuations, but no common package stating the expected interface of the WAContinuation and WAPartialContinuation classes.

A common package carrying a WAAbstractContinuation class would define a interface for all platform-specific packages to follow and make the dependencies explicit.

The work on SeasideSt/Grease#40 is related as it tries to make the code base uniform across platforms w.r.t. continuations (where thisContext is used).
@jbrichau jbrichau added this to the 3.3 milestone Sep 10, 2018
@jbrichau
Copy link
Member Author

Work started here: https://github.com/SeasideSt/Seaside/tree/issues/1007

jbrichau pushed a commit that referenced this issue Nov 24, 2018
Merge pull request #1048 from /issues/1007
7a9f2cb 
Issues/1007
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.3
Development

No branches or pull requests
1 participant
@jbrichau