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

Add Puppet validator for plans #1207

Merged
merged 1 commit into from
Feb 11, 2023
Merged

Add Puppet validator for plans #1207

merged 1 commit into from
Feb 11, 2023

Conversation

jay7x
Copy link
Contributor

@jay7x jay7x commented Jan 28, 2023

Puppet plans are silently ignored in pdk validate currently. As a Bolt user I'd like to check Bolt plan syntax before applying it.

@jay7x jay7x requested a review from a team as a code owner January 28, 2023 11:29
@jay7x
Copy link
Contributor Author

jay7x commented Jan 28, 2023

Hmm.. I'm not sure how to add this message to the pdk.pot.. I tried bundle exec rake update_pot but got all messages removed from there instead 🤔

@chelnak
Copy link
Contributor

chelnak commented Jan 29, 2023

Don't worry about the pot. We will check this out next week.

Thanks for your contribution!

@chelnak
Copy link
Contributor

chelnak commented Feb 11, 2023

@jay7x Happy to merge this! Nice work.

@chelnak chelnak merged commit a9cdb72 into puppetlabs:main Feb 11, 2023
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.

2 participants