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

puppet-syntax: Validate Hiera keys #444

Merged
merged 1 commit into from
Mar 12, 2024
Merged

Conversation

bastelfreak
Copy link
Collaborator

This is a new option in puppet-syntax 4.1.0:
voxpupuli/puppet-syntax#143

It will be enabled in puppet-syntax 5 be default:
voxpupuli/puppet-syntax#167

This is a new option in puppet-syntax 4.1.0:
voxpupuli/puppet-syntax#143

It will be enabled in puppet-syntax 5 be default:
voxpupuli/puppet-syntax#167
@bastelfreak bastelfreak requested a review from a team as a code owner March 12, 2024 11:55
@bastelfreak bastelfreak mentioned this pull request Mar 12, 2024
Copy link

codecov bot commented Mar 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.88%. Comparing base (7863f3f) to head (65ca136).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #444      +/-   ##
==========================================
+ Coverage   41.80%   41.88%   +0.08%     
==========================================
  Files          10       10              
  Lines         677      678       +1     
==========================================
+ Hits          283      284       +1     
  Misses        394      394              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@jordanbreen28 jordanbreen28 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disregard my now gone comment - had a bit of a derp moment ☕ LGTM.

@jordanbreen28 jordanbreen28 merged commit a7896d0 into puppetlabs:main Mar 12, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants