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-1758) supplement in_module_root with check for metadata.json #1154

Merged
merged 1 commit into from
Nov 29, 2021
Merged

(PDK-1758) supplement in_module_root with check for metadata.json #1154

merged 1 commit into from
Nov 29, 2021

Conversation

da-ar
Copy link

@da-ar da-ar commented Nov 29, 2021

Prior to this change the in_module_root function looked only for specific folders within the project to determine if it was at the root.
This check supplements it with a check to see if metadata.json is present enabling modules that only contain /files without looking for such a common folder name.

@da-ar da-ar requested a review from a team as a code owner November 29, 2021 11:12
@da-ar da-ar added the bug label Nov 29, 2021
Prior to this change the `in_module_root` function looked only for specific folders within the project to determine if it was at the root. 
This check supplements it with a check to see if `metadata.json` is present enabling  modules that only contain `/files` without looking for such a common folder name.
@codecov
Copy link

codecov bot commented Nov 29, 2021

Codecov Report

Merging #1154 (1e13dbf) into main (8e9271d) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1154   +/-   ##
=======================================
  Coverage   91.25%   91.25%           
=======================================
  Files         137      137           
  Lines        5580     5580           
=======================================
  Hits         5092     5092           
  Misses        488      488           
Impacted Files Coverage Δ
lib/pdk/util.rb 100.00% <100.00%> (ø)

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 aeedce7...1e13dbf. Read the comment docs.

@sanfrancrisko sanfrancrisko merged commit cc24a00 into puppetlabs:main Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants