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 default facter version from rspec-puppet-facts #49

Merged

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Aug 10, 2021

Since rspec-puppet-facts 2.0.0 the default Facter version is derived from the Puppet version. There is no more need to do this locally. It also handles Puppet 7 which our code doesn't.

I should note I didn't test this yet.

Since rspec-puppet-facts 2.0.0 the default Facter version is derived
from the Puppet version. There is no more need to do this locally. It
also handles Puppet 7 which our code doesn't.
@codecov
Copy link

codecov bot commented Aug 10, 2021

Codecov Report

Merging #49 (91f7a93) into master (de36f67) will not change coverage.
The diff coverage is n/a.

❗ Current head 91f7a93 differs from pull request most recent head bcffa1c. Consider uploading reports for the commit bcffa1c to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   25.75%   25.75%           
=======================================
  Files           2        2           
  Lines          66       66           
=======================================
  Hits           17       17           
  Misses         49       49           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de36f67...bcffa1c. Read the comment docs.

@bastelfreak bastelfreak merged commit a8c39cd into voxpupuli:master Aug 10, 2021
@bastelfreak
Copy link
Member

ah sorry for the early merge, I realized the I didn't test this yet while pressing the button. let me know if we should revert it.

bastelfreak added a commit to bastelfreak/puppet-borg that referenced this pull request Aug 11, 2021
@bastelfreak
Copy link
Member

tested it in voxpupuli/puppet-borg#118

bastelfreak added a commit to bastelfreak/puppet-borg that referenced this pull request Aug 13, 2021
bastelfreak added a commit to voxpupuli/puppet-borg that referenced this pull request Aug 13, 2021
bastelfreak added a commit to voxpupuli/puppet-borg that referenced this pull request Aug 13, 2021
bastelfreak added a commit to voxpupuli/puppet-borg that referenced this pull request Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants