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

Recognition of check_strings/only_variable_string.rb #72

Open
promarcel opened this issue Nov 2, 2022 · 0 comments
Open

Recognition of check_strings/only_variable_string.rb #72

promarcel opened this issue Nov 2, 2022 · 0 comments
Labels
bug Something isn't working community

Comments

@promarcel
Copy link

Describe the Bug

When checking for only variables in strings, there is a case where strings are not recognized even though they should be.

Only variables in strings formatted as follows are recognized correctly: "${variable}".
Unfortunately, however, the following format is not recognized: "${ variable }". Detail on the spaces after and before the curly bracket.

It is located inside the check check_strings/only_variable_string.rb.

Expected Behavior

The above format should also trigger the warning messages, even if Puppet interprets this notation correctly in manifests it should not be used as “best practice”.

Steps to Reproduce

Using "${ variable }" inside a Puppet manifest.

Environment

  • Version: Latest from RubyGems (3.0.1)
  • Platform: Ubuntu 22.04

Additional Context

No additional information provided. Unfortunately, I am not that familiar with Ruby to implement a fix for this by myself.

@promarcel promarcel added the bug Something isn't working label Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working community
Projects
None yet
Development

No branches or pull requests

2 participants