Skip to content
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

(GH-284) Added support for #region folding #285

Merged
merged 1 commit into from
Jun 12, 2018
Merged

(GH-284) Added support for #region folding #285

merged 1 commit into from
Jun 12, 2018

Conversation

neitik
Copy link
Contributor

@neitik neitik commented Jun 7, 2018

Hello, as I mentioned in #284, I would really like to see #region folding support for puppet manifests, this change adapted from microsoft/vscode-python#33 should handle that.

@glennsarti
Copy link
Contributor

Before:
image

After:
image

We actually got folding "for free" due to the puppet manifest styleguide. But this seems fine.

Thanks for the PR @neitik.

Copy link
Contributor

@glennsarti glennsarti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@glennsarti
Copy link
Contributor

@jpogran or @michaeltlombardi You happy for merge?

@glennsarti glennsarti added this to the 0.11.0 milestone Jun 8, 2018
@jpogran
Copy link
Contributor

jpogran commented Jun 12, 2018

Thanks for the code @neitik ! LGTM! really appreciate your contribution.

@jpogran jpogran merged commit 3eeeb99 into puppetlabs:master Jun 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants