atom / language-puppet

Puppet package for Atom
Other
36 stars 31 forks source link

Formatting based on style guides #10

Closed ferventcoder closed 8 years ago

ferventcoder commented 9 years ago

I don't see that this does this, but if it does feel free to close this. Would be nice to have a key combo that would automatically format the manifest.

jacobbednarz commented 9 years ago

I don't see why this wouldn't be possible but there would need to be a lot of smarts built in. Example, while single quotes is recommended you would need to ensure you don't change double quotes to single quotes if the string is interpolated with a variable.

I personally use rodjek/puppet-lint so perhaps we could leverage that here?

ferventcoder commented 9 years ago

Yes, definitely leverage existing tools.

winstliu commented 8 years ago

Atom's language packages are meant to only provide language highlighting - I would recommend installing a different package for formatting and linting.