Skip to content

Conversation

@glennsarti
Copy link
Contributor

Fixes #289

Previously only files that were less than 4KB in size could use the
format-on-type feature. This commit updates the Language Server to honor a
new configuration setting puppet.editorService.formatOnType.maxFileSize which
will determine the maximum file size. A size of 0 will disable the size check.
The default is 4096 as per the previous behaviour.

No specific tests have been added and has been manually tested.

@glennsarti glennsarti requested a review from jpogran as a code owner January 13, 2021 07:06
@glennsarti glennsarti added this to the 1.0.2 milestone Jan 13, 2021
@glennsarti glennsarti self-assigned this Jan 13, 2021
Previously only files that were less than 4KB in size could use the
format-on-type feature. This commit updates the Language Server to honor a
new configuration setting puppet.editorService.formatOnType.maxFileSize which
will determine the maximum file size.  A size of 0 will disable the size check.
The default is 4096 as per the previous behaviour.

No specific tests have been added and has been manually tested.
@glennsarti glennsarti force-pushed the gh289-format-file-size branch from 3e08c07 to b8a3386 Compare January 13, 2021 07:35
@jpogran jpogran merged commit f9b2fe2 into puppetlabs:main Jan 13, 2021
@glennsarti glennsarti deleted the gh289-format-file-size branch January 14, 2021 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Config setting for format on type file size limit

2 participants