[aerogear-dev] JavaScript formatter

[Lucas Holmquist]

On Oct 29, 2013, at 7:14 AM, Matthias Wessendorf <matzew at apache.org> wrote:

> 
> 
> 
> On Tue, Oct 29, 2013 at 12:08 PM, Erik Jan de Wit <edewit at redhat.com> wrote:
>>  
>> Talking with qmx and lholmquist about this it seems that JS land doesn't have good tools for this. 
>> 
>> 
>> hrm - the grunt worked for me; but I don't know if it is _that_ good

grunt will run jshint, which doesn't do the formatting for you,  but tells you what you need to fix.  but until up to a point.  

I think there is a difference between formatting and whitespace


> 
> The formatter that is available with grunt seems to have very little settings [1], but if that is fine for most of us by all means we should use it!
> 
> [1] https://npmjs.org/package/grunt-jsbeautifier
> 
> perhaps here we could help/improve ? 

+1 to this option

>  
> 
>>  
>> 
>> To have something in the mean time we could use something like the eclipse formatter, because it can format JS has a ton of options and there are already a lot of external tools that use it:
>> 
>> https://github.com/krasa/EclipseCodeFormatter
>> 
>> I doubt that a JS weenie will use Eclipse/IDEA
> 
Correct,  none of us "Rock Stars" would use an IDE :)


> It's meant as an example you don't need the full eclipse IDE to use the formatting tool. It can be used standalone and it has a ton of options about where to put whitespace. We could also build our own javascript code formatter, but why not have something that works today. And I think even a javascript coder can execute a command line utility.

if there is a java dependency then it is a no go for JS land

> 
> Just trying to be pragmatic 
> 
> 
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> 
> 
> -- 
> Matthias Wessendorf 
> 
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20131029/ecff2a3f/attachment.html