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

Status on async / await straight through compilation? #977

Closed
ozra opened this issue Sep 19, 2017 · 8 comments
Closed

Status on async / await straight through compilation? #977

ozra opened this issue Sep 19, 2017 · 8 comments

Comments

@ozra
Copy link

ozra commented Sep 19, 2017

I know there is a number of issues on this and related, I just post this to get a fresh up to date perspective.

I'm back to some contract nodejs-coding for the first time in a few years of mainly systems coding. Now it seems Koa and similar frameworks are established enough to be usable. So of course I want to do that. And of course I want to code in LiveScript.

So, what gives?

  • Will bonafide (no "magic" compilation) async and await keywords come in main distro LiveScript soon?
  • Otherwise, which fork is best maintained (following gkz/) with the feature, any consensus in the LS-community?
@vendethiel
Copy link
Contributor

vendethiel commented Sep 19, 2017

#836 is interesting, but has no tests. The PR should still work, since lsc doesn't move very fast.
You'd need @rhendric 's approval though.

Doesn't co work with yield though? Did Koa switch to full async/await?

Otherwise, which fork is best maintained (following gkz/) with the feature

I only know of @summivox 's fork

any consensus in the LS-community?

I don't think that's active enough to reach such a consensus.

@ozra
Copy link
Author

ozra commented Sep 19, 2017

Thanks for the intell @vendethiel! I've just started gathering intell on what techs to use for the nodejs part.

LS is a given, even though I will most likely compile it down to JS further down the line for other developers to maintain.

From what I've quickly gleaned Koa2 seems to go the async / await road.

I'm gonna evaluate available options a bit more before nailing down the roadmap.

Thanks again!

@ozra
Copy link
Author

ozra commented Sep 19, 2017

I can only hope #836 will end up in a release soon.

Is lack of tests and documentation what hinders it from happening?

For now I'm trying out the livescript-async npm uploaded by @gkovacs.

@rhendric
Copy link
Collaborator

Tests at least. (I don't have a plan for dealing with LS documentation issues in general; the website hasn't been updated since the 1.4 release and I haven't looked into what it would take to bring it up to date.)

I haven't reviewed the code in that PR yet; I'm waiting for its author to add tests and tell me that it's ready for a review, since it's been a work in progress for so long. If review turns up any issues, obviously someone will have to deal with those.

@summivox
Copy link
Contributor

summivox commented Sep 19, 2017 via email

@rhendric
Copy link
Collaborator

I don't have a lot of disposable time, myself. Someone who feels strongly about having this feature in official LiveScript should pick it up in this case.

@gkovacs
Copy link
Contributor

gkovacs commented Sep 20, 2017

@rhendric I'm happy to pick this up as I've been using this feature in production for several months now, I've made a new pull request containing tests at #978 please review when you have time, thank you.

@rhendric
Copy link
Collaborator

#978 has landed!

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

5 participants