-
Notifications
You must be signed in to change notification settings - Fork 2
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
Publish as package to npm for reusability #7
Comments
Good morning @Glavin001. I did read the article "How to add a Beautifier for a Language" a couple month ago. i forked your atom-beautify for adding jade support. and i tried. but as a newbie, it was too diffcult to do. With my whole heart, i want to contribute community if i can. but it will take many time to find out how atom-beautify work and to run coffeescript. If there are a short way to help you, tell me. tks. |
Don't worry about that Atom Beautify integration -- I will take care of that for you 😃 . If you could publish a package that implemented jade beautification then I will make the appropriate changes to Atom Beautify. Suggested usage for jade beautify package: var jadeBeautify = require('jade-beautify');
var code = "...";
var options = {
indent_size: 4
};
try {
var output = jadeBeautify(code, options);
} catch (error) {
// Error occurred
} Great work so far. Thanks in advance! |
Ok @Glavin001. Did you mean that i write a new npm package for atom-beautify. if yes, i will. and how about changing it's name to 'beautify-jade' without confusing. |
Yes, to publish to npm, (see https://docs.npmjs.com/getting-started/publishing-npm-packages for details) however not necessarily specifically for Atom Beautify.
Atom packages are published with apm while Node.js packages are published to npm. So please feel free to keep the the name as Thank you. |
Ok. i understand you clearly. but is it possible that a module support both a npm node moudle and a atom package simultaniously ? there's only one entry-porint 'main' on 'package.json' file. |
It would probably be easiest if you created a separate package to publish to npm. |
That's what i say it making confusing. |
And FYI, Jade change it's project name to 'Pug'. |
Good eye! I did not know that. Thanks for sharing. |
Good Morning @Glavin001 . I'm very happy to release a pug-beautify on npm. First i modified the jade-beautify for monitoring whether there's error. Check that module what you want. Enjoy it. Thanks. |
If you publish this package I can add support for it on Atom Beautify. See Glavin001/atom-beautify#199
Thank you and great work!
The text was updated successfully, but these errors were encountered: