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

(maint) load rake tasks from optional libraries #279

Merged
merged 1 commit into from
Feb 19, 2019

Conversation

DavidS
Copy link
Contributor

@DavidS DavidS commented Feb 15, 2019

This removes the need for the same loading code in pdk-templates at
https://github.com/puppetlabs/pdk-templates/blob/7281db56ead0feeacbc70e5d5dfb3bb885c554ba/moduleroot/Rakefile.erb#L13-L16

Note that puppet-syntax/tasks/puppet-syntax was already loaded in this
file further down.

This removes the need for the same loading code in pdk-templates at
https://github.com/puppetlabs/pdk-templates/blob/7281db56ead0feeacbc70e5d5dfb3bb885c554ba/moduleroot/Rakefile.erb#L13-L16

Note that `puppet-syntax/tasks/puppet-syntax` was already loaded in this
file further down.
@DavidS
Copy link
Contributor Author

DavidS commented Feb 15, 2019

This supercedes #274

@codecov-io
Copy link

codecov-io commented Feb 15, 2019

Codecov Report

Merging #279 into master will increase coverage by 0.42%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
+ Coverage   44.45%   44.88%   +0.42%     
==========================================
  Files          11       11              
  Lines         776      782       +6     
==========================================
+ Hits          345      351       +6     
  Misses        431      431
Impacted Files Coverage Δ
lib/puppetlabs_spec_helper/rake_tasks.rb 47.86% <100%> (+1.52%) ⬆️

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 8ccffc7...30957d9. Read the comment docs.

@puppetcla
Copy link

CLA signed by all contributors.

@tphoney tphoney merged commit 90f4c0e into puppetlabs:master Feb 19, 2019
@DavidS DavidS deleted the optional-gem-tasks branch May 2, 2019 12:52
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.

5 participants