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

Architecture #23

Open
danielbayley opened this issue Jun 9, 2016 · 3 comments
Open

Architecture #23

danielbayley opened this issue Jun 9, 2016 · 3 comments

Comments

@danielbayley
Copy link
Owner

I feel like this should be a plugin to some kind of atom-transpiler base package. I say this especially as I have another idea to put an Atom wrapper around decaf for transpiling CoffeeScript to ES6 in the opposite direction, or maybe that should be another source-preview package…
@jdcrensh Any thoughts on this?

@jdcrensh
Copy link
Collaborator

jdcrensh commented Jun 9, 2016

I like the idea, but I'm not sure how receptive existing users would be to installing an extra package dependency in order to keep this one working. Does Atom have any sort of mechanism for easing that situation? I'd still like to see how this base package would work, once there's something to look at.

@danielbayley
Copy link
Owner Author

I like the idea, but I'm not sure how receptive existing users would be to installing an extra package dependency in order to keep this one working.

One of the main benefits of this approach will be standardising how to transpile things in Atom, and making it easier to write plugins to do so. Also organising all transpilers under a single transpile [context]/menu as it gets way too cluttered already.

Does Atom have any sort of mechanism for easing that situation?

I'm not sure about that, there are some open PRs on this kind of thing already though, and also atom-package-deps. On a related note, I deprecated one of my packages recently with this code.

Iv'e already written half of both packages, so I'll link to them here for you to look at once they're up.

Hopefully then maybe I can add this along with a bunch of other existing Atom transpiler packages as plugins.

@danielbayley
Copy link
Owner Author

I'd still like to see how this base package would work, once there's something to look at.

@jdcrensh I finally got around to implementing this, and so here is that base package… atom-transpile.

I also managed to finish a transpile-decaf plugin, to give an example.

Also see #25, and a similar PR on another existing package here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants