issues Search Results · repo:voxpupuli/puppet-lint-trailing_comma-check language:Ruby
Filter by
12 results
(61 ms)12 results
invoxpupuli/puppet-lint-trailing_comma-check (press backspace or delete to remove)Describe the Bug
if you have these 2 errors next to each other it will crash, if you manaualy fix just one of them it work fine
ERROR: there should be a single space or newline before a closing brace ...
zombiedk
- Opened on Mar 27, 2023
- #32
Since pdk 2.5 it appears that the autocorrect runs twice for example running a pdk validate -a on the following code:
class woof {
Service { httpd :
ensure = running,
enable = true,
}
...
davidsandilands
- 6
- Opened on Feb 23, 2023
- #31
Would expect this to throw a warning as it is missing the comma at the end of line 2.
class foo (
String $bar = asdf
) {
ghoneycutt
- Opened on Dec 18, 2018
- #18
Running this check with --fix produced
diff --git a/dist/account/manifests/user.pp b/dist/account/manifests/user.pp
index c1e17f13b..58ab923cb 100644
--- a/dist/account/manifests/user.pp
+++ b/dist/account/manifests/user.pp ...
skip-changelog
genebean
- 5
- Opened on Oct 24, 2018
- #16
Per the comment here https://github.com/voxpupuli/puppet-lint-trailing_comma-check/issues/12#issuecomment-431610509
interpolated heredocs aren t being handled correctly.
skip-changelog
seanmil
- Opened on Oct 22, 2018
- #14
Code
concat::fragment { /etc/amavis/conf.d/50-user/dkim :
target = /etc/amavis/conf.d/50-user ,
order = 10 ,
content = @(FILE)
$signed_header_fields{ received } = 0;
$enable_dkim_verification ...
skip-changelog
jkroepke
- 3
- Opened on Oct 21, 2018
- #13
When I let this check fix the issues it finds it breaks all my heredocs by adding a comma to the closing tag. For
example:
$ bundle exec puppet-lint site dist site.pp --fix --only-checks trailing_comma ...
skip-changelog
- 6
- Opened on Oct 20, 2018
- #12
Version used: 0.3.2
When a Puppet-language function specifies a return data type that is more than a single namespace, there is a false
positive.
Code that causes a false positive at: $_return = /foo/bar ...
skip-changelog
natemccurdy
- 2
- Opened on Oct 8, 2018
- #11
Currently this puppet-lint rule complains about the following syntax:
file { /tmp/test.txt :
ensure = file ,
content = @(EOT),
Hello
World
EOT
}
Adding a comma after the trailing heredoc ...
skip-changelog
joshuaspence
- 1
- Opened on Oct 6, 2017
- #8
The following is a false positive:
class test {
package {
[ foo ]:
require = [
File[
/etc/foo.conf ,
/etc/bar.conf
],
Packages[
...
skip-changelog
cbowman0
- Opened on Mar 27, 2017
- #5
Learn how you can use GitHub Issues to plan and track your work.
Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub IssuesProTip!
Press the /
key to activate the search input again and adjust your query.Learn how you can use GitHub Issues to plan and track your work.
Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub IssuesProTip!
Restrict your search to the title by using the in:title qualifier.