Skip to content

config: Find YAML files recursively #2860

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pawiecz
Copy link
Contributor

@pawiecz pawiecz commented Apr 25, 2025

Previously only top-level files were found in a given directory.

Fixes: #2859

Copy link
Member

@kwilczynski kwilczynski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! A small nit, but it's non-blocking, for sure.

@kwilczynski
Copy link
Member

The following change should, once this branch is rebased, help resolve the CI linker failure:

@pawiecz pawiecz force-pushed the find-yaml-files-recursively branch from 4aab0ae to 2271024 Compare April 28, 2025 11:39
@nuclearcat
Copy link
Member

@pawiecz can you rebase your PR, so probably latest workflows will be used for checks

@nuclearcat nuclearcat added the staging-skip Don't test automatically on staging.kernelci.org label Apr 29, 2025
@pawiecz pawiecz force-pushed the find-yaml-files-recursively branch 2 times, most recently from 7f548b8 to e210c87 Compare April 30, 2025 11:29
Previously only top-level files were found in a given directory.

Fixes: kernelci#2859

Signed-off-by: Paweł Wieczorek <[email protected]>
@pawiecz pawiecz force-pushed the find-yaml-files-recursively branch from e210c87 to 4a03e77 Compare April 30, 2025 11:30
@pawiecz pawiecz removed the staging-skip Don't test automatically on staging.kernelci.org label Apr 30, 2025
Copy link
Member

@kwilczynski kwilczynski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Loading YAML configs works only for files found in top-level config_path
3 participants