-
Notifications
You must be signed in to change notification settings - Fork 85
Add buff-ignore; correctly apply chefignore #192
Conversation
# @return [Boolean] | ||
# true if it should be ignored, false otherwise | ||
def ignored?(file) | ||
!!chefignore && chefignore.send(:ignored?, file) |
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.
why is this a private function on the Chefignore?
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.
It can be made public, but the ignore file should really only respond to #apply
, given an array argument. I can make the method public if that makes more sense though.
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.
I think it makes sense to make the Chefignore class a bit more flexible
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.
Fixed
@sethvargo the build is still failing on this branch |
@reset right, we talked about it in code review. I don't have a sustainable way to test this and needed some pointers. The order for SHAs is not guaranteed |
@reset I had to call |
Add buff-ignore; correctly apply chefignore
@reset 👀