-
Notifications
You must be signed in to change notification settings - Fork 682
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
Metadata #149
Metadata #149
Conversation
@@ -39,7 +52,7 @@ def get_filenames(paths) | |||
end | |||
|
|||
HANDLERS = [ | |||
ChefAuditDir, ServerspecDir, FlatDir | |||
ProfileDir, ServerspecDir, FlatDir |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there renaming typo? ProfileDir
vs ChefAuditDir
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catch, I missed to add ChefAuditDir! Thank you @chris-rock
This PR has passed 'Verify' and is ready for review and approval! |
@@ -15,16 +15,17 @@ def resolve(target) | |||
# find all files in the folder | |||
files = Dir[File.join(target, '**', '*')] | |||
# remove the prefix | |||
files = files.map { |x| x[target.length + 1..-1] } | |||
files = files.map { |x| x.sub(target, '') } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
way better. Thanks @arlimus
Thanks @arlimus for this improvement. |
@delivery approve |
Merged change 1d1b403b-c61e-4a7d-8445-b90a20e9e464 From review branch metadata into master Signed-off-by: chartmann <chartmann@chef.io>
Change: 1d1b403b-c61e-4a7d-8445-b90a20e9e464 approved by: @chris-rock |
This PR has passed 'Acceptance' and is ready to be delivered! |
The auto-include libraries is about providing library functions to all tests within scope:
Here, crypto_lib will be included by default, sothat crypto_test can use it.