puppetlabs / puppet-syntax-emacs

Puppet language syntax highlighting for Emacs
Other
25 stars 17 forks source link

Fix #1889 - Align fat arrows #14

Closed dgellow closed 5 years ago

dgellow commented 10 years ago

Fix PUP-1889.

Format resource's attributes according to the style guide, section 9.2 "Arrow Alignment".

Current behaviour

I have a resource Exec['test']. If I indent the code (with for exemple indent-region), nothing happen.

    exec { 'test':
      subscribe => File['/etc/test'],
      refreshonly => true, | ;; cursor is here
    }

Hit Tab, or select resource and call indent-region. Nothing change.

New behaviour

Same resource, attributes are formatted with Tab, C-m, C-j, indent-region, etc.

    exec { 'test':
      subscribe => File['/etc/test'],
      refreshonly => true, | ;; <- cursor is here
    }

Hit Tab. Attributes are formatted.

    exec { 'test':
      subscribe   => File['/etc/test'],
      refreshonly => true, | ;; <- cursor is here
    }
puppetcla commented 10 years ago

Waiting for CLA signature by @dgellow

@dgellow - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppetlabs.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppetlabs.com/community/trivial_patch_exemption.html

dgellow commented 10 years ago

@puppetcla done

puppetcla commented 10 years ago

CLA signed by all contributors.

ahpook commented 5 years ago

superseded by voxpupuli/puppet-mode