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

Use new puppet-lint gem #15

Closed
wants to merge 4 commits into from
Closed

Use new puppet-lint gem #15

wants to merge 4 commits into from

Conversation

GSPatton
Copy link

Using the now-maintained puppetlabs-puppet-lint over puppet-lint

@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (96cf8ae) 100.00% compared to head (f22bac5) 100.00%.

❗ Current head f22bac5 differs from pull request most recent head bb25904. Consider uploading reports for the commit bb25904 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #15   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines           67        67           
=========================================
  Hits            67        67           

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

@GSPatton GSPatton closed this Nov 8, 2023
@GSPatton GSPatton reopened this Nov 8, 2023
@GSPatton GSPatton closed this Nov 15, 2023
@GSPatton GSPatton reopened this Nov 15, 2023
@@ -24,6 +24,6 @@
]
end

require 'puppet-lint'
require 'puppetlabs-lint'

PuppetLint::Plugins.load_spec_helper
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this also be updated to have its own namespace? Otherwise you still have the problem of overlapping with the old gem.

@ekohl
Copy link
Member

ekohl commented Nov 25, 2023

This can now be closed, right?

@ekohl ekohl closed this Dec 27, 2023
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.

2 participants