Skip to content
This repository has been archived by the owner on Oct 28, 2019. It is now read-only.

Replace deasync, inquirer, or both #13

Closed
MathieuAA opened this issue Jun 24, 2015 · 2 comments
Closed

Replace deasync, inquirer, or both #13

MathieuAA opened this issue Jun 24, 2015 · 2 comments
Assignees

Comments

@MathieuAA
Copy link
Member

The deasync package must either be replaced, or a fix must be found.

See #12 for more details.

@MathieuAA MathieuAA changed the title Replace of fix deasync Replace or fix deasync Jun 24, 2015
@MathieuAA
Copy link
Member Author

Apparently, using NodeJS on Windows can be troublesome (some tools may not be installed, etc.).
This link may help.

@MathieuAA MathieuAA changed the title Replace or fix deasync Replace deasync, inquirer, or both Aug 1, 2015
@MathieuAA MathieuAA self-assigned this Aug 1, 2015
@MathieuAA
Copy link
Member Author

Update and end: instead of replacing, we could just not use any of them (inquirer and deasync), and still keep them in the code.
This is the main idea: we could just not require the user for his/her input, and just pick the one editor we can't detect (right now it's UML Designer). If, however, we decide to add another editor that can't be detected either, then the user would have to pick the right one with inquirer (thus the call to deasync).
Better safe than sorry.
It's currently being maintained in a separate branch that wil, hopefuly, be merged ASAP.

It's not a perfect solution, but we hope in the future that the great JDL and the soon-to-be UML editor will replace the editors out there.

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

No branches or pull requests

1 participant