Update CI workflow paths and adjust concurrency filters#2016
Merged
Dashing-Nelson merged 1 commit intojacques/ecdata-824-move-core-entities-code-to-the-matrix-repo_2from Jan 9, 2026
Conversation
Dashing-Nelson
merged commit 931389e
into
jacques/ecdata-824-move-core-entities-code-to-the-matrix-repo_2
1 check passed
Dashing-Nelson
deleted the
nelson/aip-696-decide-on-branch-protection-rule-or-ci-merging-strategy
branch
Dashing-Nelson
added a commit
that referenced
this pull request
Jan 9, 2026
* Add core entities with integration test working * bump * Revert run comparison parameters * Fix disease labels * Add github actions * bump * Update working directory * Update paths * Add PAT again * bump * Update matrix utils * Update pipelines/core_entities/scripts/compare_releases.py * Delete models * PDate readme * Refactor drug list pipeline * Add test settings for core entities run pipeline * Bump to disease list * Switch checkout and PAT * Replace google cloud vars with matrix style * Switch PAT action order * Remove test suite * bump * Bump * Update core_entities_run_pipeline.yaml * Update CI workflow paths and adjust concurrency filters (#2016) --------- Co-authored-by: everycure <123398416+everycure@users.noreply.github.com> Co-authored-by: Nelson Alfonso <45660392+Dashing-Nelson@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the changes
This pull request updates the configuration for the matrix CI workflow and updates the
infra/secretssubmodule. The main focus is on how the workflow triggers and filters files for running CI jobs, making the workflow more maintainable and less prone to unnecessary runs.Workflow configuration improvements:
pathsfilters from thepull_requestandpushtriggers in.github/workflows/matrix-ci.yml, which means the workflow will now trigger on all changes to the relevant branches or PR events, not just when specific files are modified.filters.srclist in the job configuration to include additional relevant files and directories (libs/**,docker-composefiles), ensuring the job only runs when these files change, thus keeping the workflow efficient.Submodule update:
infra/secretssubmodule to a new commit, bringing in the latest changes to secrets management.Fixes / Resolves the following issues:
Checklist:
enhancementorbug)pulling in latest main, uncomment the below "Merge Notification" section and
describe steps necessary for people
kedro run -e sample -p test_sample(see sample environment guide)