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 PureScript implementation #573

Closed
wants to merge 12 commits into from
Closed

Conversation

mrsekut
Copy link
Contributor

@mrsekut mrsekut commented Jun 20, 2021

This is a PureScript implementaion.
The test of step5 has not passed (implementing..)
Dockerfile etc. are not ready yet.

@kanaka
Copy link
Owner

kanaka commented Jul 12, 2021

@mrsekut any progress on step5? I'm going to hold off on reviewing it until that is working since it could lead to refactoring of other steps.

@mrsekut
Copy link
Contributor Author

mrsekut commented Jul 12, 2021

I'm sorry, please wait a little longer.

I learned that PureScript needs special measures to mitigate stack overflow, so I will implement it.

Also, the test in step A has failed, and that problem has been fixed, but I haven't pushed it yet. After fixing step 5, let me talk about how to fix the commit log.

@mrsekut
Copy link
Contributor Author

mrsekut commented Jul 24, 2021

@kanaka
It took a while, but I was able to fix it. The content of the correction was large, so I recreated the PR. #581

@mrsekut mrsekut closed this Jul 24, 2021
@mrsekut mrsekut deleted the ft-mrsekut-purs branch July 24, 2021 05:06
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.

None yet

2 participants