chart-testing feat: evaluate .ignore files during the change detection process

What this PR does / why we need it: This PR adds the possible to exclude files during the change detection process. This can be particularly useful in the context of the version increment check. If the feature is enabled, all .ctignore & .helmignore files inside the configured chart directories are evaluated by default and applied relative to the file path. All common glob patterns including the globstar pattern ** are supported. In addition, the config parameter considered-dotignore-files can be adjusted to specify which .ignore files are to be considered.

Added config parameters (default values):

evaluate-dotignore-files: false
considered-dotignore-files:
    - .ctignore
    - .helmignore

Special notes for your reviewer: I added/modified test cases to cover the new functionality.

Signed-off-by: Philipp Waller [email protected]

Mar 15 '22 15:03 philippwaller

Dear Core Team, do you have any feedback or change request regarding this PR? Thanks!

Mar 28 '22 09:03 philippwaller

Any updates on this? I would like to see this functionality, but seems like there was no reaction from the maintainers?

Apr 05 '23 12:04 johanneskastl

I would be happy to revise the PR if the code is considered for future releases. Just let me know and I will make the necessary changes to ensure that the code is ready for integration.

Apr 07 '23 08:04 philippwaller

This looks good, can we rebase and get this PR up-to-date, so we can run some manual tests? also will be good if you have a sample ci pipeline to show this working

thanks for working on this