Skip to content

Filter matrix to directories only - #224

Closed
mendral-app[bot] wants to merge 1 commit into
mainfrom
mendral/filter-matrix-directories-only
Closed

Filter matrix to directories only#224
mendral-app[bot] wants to merge 1 commit into
mainfrom
mendral/filter-matrix-directories-only

Conversation

@mendral-app

@mendral-app mendral-app Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix the generate-matrix step to only include directories from hub/ in the build matrix, preventing files like node-test.py from causing Docker buildx failures on tag pushes.

Changes

Updated the jq filter in .github/workflows/build.yaml:260 from:

map(select(.name) | .name)

to:

map(select(.type == "directory") | .name)

This ensures tree -J output is filtered by entry type, so only actual directories are passed to the Docker build context.

Related


Note

Created by Mendral. Tag @mendral-app with feedback or questions.

The jq filter now checks .type == "directory" instead of just .name,
preventing non-directory entries like hub/node-test.py from being
included in the build matrix and causing Docker buildx failures.
@mendral-app
mendral-app Bot requested a review from a team June 11, 2026 21:08
@mendral-app
mendral-app Bot marked this pull request as ready for review June 11, 2026 21:15
@cploujoux

Copy link
Copy Markdown
Contributor

Closing as stale.

This PR has been in merge conflict and unreviewed for a long time. Checked before closing: none of the packages it bumps has an open Dependabot alert on this repo today — so it is not holding back any security fix, it is only adding noise to the review queue.

Where a real vulnerability does remain open, it is being handled by a dedicated remediation PR instead of a stale bulk update.

@cploujoux cploujoux closed this Aug 14, 2026
@cploujoux
cploujoux deleted the mendral/filter-matrix-directories-only branch August 14, 2026 22:57
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.

1 participant