run_command: support array of commands
Nobody has claimed this yet.
- Dominant language
- Ruby
- Stars
- 558
- Forks
- 224
- Avg merge
- 1h 4m
- Merged PRs (30d)
- 1
Description
Use Case
Due to security concerns, the best practice for commands in Puppet DSL are now arrays: https://www.puppet.com/docs/puppet/7/types/exec.html#exec-attribute-command
The run_command() only supports the String datatype as input and I think it makes sense that it supports array as well.
Describe the Solution You Would Like
Array support for the commands in run_command()
Describe Alternatives You've Considered
A clear and concise description of any alternative solutions or features you've considered.
Additional Context
Add any other context or screenshots about the feature request here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the run_command() entry point and reading how it currently validates command input. Trace the existing command path and its tests, then add coverage showing the requested array form is accepted while existing string behavior remains intact. Done means run_command() supports arrays of commands and the relevant tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ruby
- Domain
- devops
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100