-
Notifications
You must be signed in to change notification settings - Fork 12
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
MantraJS-atom vs Mantra-CLI #28
Comments
Maybe the atom-package could use mantrajs-cli under the hood? Can't say much about the 3 disagreements though. I think the mantra-sample application is the best guideline here |
If you read docs, the mantra atom package is fully configurable via yaml file, whatever you do not like can be changed with no hassle. You can change file names, file contents, directories, even add or reorganise panes. This is not possible with cli package. If you feel that these changes would benefit the whole community, just create a PR, like the recent one which changed stories folder to .stories folder. |
@tomitrescak , yeah the atom-package's yaml configs are great! Would be cool if the cli could use the same templates. @sungwoncho 's idea of having a .mantra-config-folder sounds also promising: https://talk.mantrajs.com/t/do-we-have-to-use-the-mantra-generated-eslintrc/178/3 |
Hi,
It appears that MantraJS-atom and Mantra-CLI are in disagreement about a few things.
index.js
, while mantra-cli createsmain.js
in server and client folders.Using both Mantra-atom and Mantra-CLI therefore causes duplication errors for methods and publications in meteor server.
unable to resolve
type errors), whereas Mantra CLI creates and requires ./configs/contextThere might be other incongruencies.
Please advise on what we should do. :)
PS. Thanks for two awesome tools. My life has improved <3
The text was updated successfully, but these errors were encountered: