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

(PDK-1137) Determine module name from metadata when possible #265

Merged
merged 1 commit into from
Nov 26, 2018

Conversation

rodjek
Copy link
Contributor

@rodjek rodjek commented Nov 26, 2018

When creating the automatic symlink fixture, determine the module name from the contents of metadata.json if possible and fall back to looking at the name of the working directory. This clears up the error reported in PDK-1137 where appveyor was converting the underscores in the repo name into dashes when cloning.

@codecov-io
Copy link

codecov-io commented Nov 26, 2018

Codecov Report

Merging #265 into master will increase coverage by 0.62%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #265      +/-   ##
==========================================
+ Coverage   40.34%   40.97%   +0.62%     
==========================================
  Files          10       10              
  Lines         751      759       +8     
==========================================
+ Hits          303      311       +8     
  Misses        448      448
Impacted Files Coverage Δ
lib/puppetlabs_spec_helper/tasks/fixtures.rb 34.08% <88.88%> (+2.45%) ⬆️

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 ef83ccd...b62449b. Read the comment docs.

@puppetcla
Copy link

CLA signed by all contributors.

@bmjen bmjen merged commit 879ce61 into puppetlabs:master Nov 26, 2018
@rodjek rodjek deleted the pdk-1137 branch November 26, 2018 23:27
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.

6 participants