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

Convert to JavaScript? #147

Open
tautologico opened this issue Feb 11, 2019 · 4 comments
Open

Convert to JavaScript? #147

tautologico opened this issue Feb 11, 2019 · 4 comments

Comments

@tautologico
Copy link

I realize this is not urgent and the package works as is, but I think the recommended way to create atom packages now is to use JavaScript/ES6 instead of CoffeeScript. Having the code in JS instead of CS has the advantage that many more people are familiar with JS than with CS (my case), besides the other network effects of using a more popular language. Also ES6 incorporated many of the improvements brought by CS.

As a way to familiarize myself with the code I'm willing to convert it to JS and send PRs, but only if the maintainers are interested in this conversion.

@oggy
Copy link
Collaborator

oggy commented Feb 13, 2019

Gah, javascript is so much less fun than coffeescript... 😝

This seems inevitable, but right now is not a great time, as I have a fairly busy branch implementing isearch right now (feel free to try it!). Perhaps after that's merged?

@tautologico
Copy link
Author

Fair enough, I will try to test the search branch.

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

3 participants
@oggy @tautologico and others