Skip to content

Take the whole _resolved block instead of ten own-* inputs - #55

Merged
mcocdawc merged 1 commit into
mainfrom
dep-table-takes-resolved
Sep 10, 2026
Merged

mcocdawc merged 1 commit into
mainfrom
dep-table-takes-resolved

Conversation

@mcocdawc

@mcocdawc mcocdawc commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Ten own-* lines per call site, six identical copies across eckit/eccodes/ecflow, all
saying "read this field out of matrix._resolved". Now just:

- uses: ecmwf/ci-infrastructure/actions/print-dep-table@main
  with:
    resolved: ${{ toJSON(matrix._resolved) }}
  • resolve_deps puts own-name into _resolved, so a workflow no longer restates its own
    package name — and a ci.yml copied between repos can't still say own-name: eckit.
  • print_dep_table takes --resolved; the own-* → column mapping lives in one tested dict
    instead of six workflows.
  • action.yml: 13 inputs → 4. The inline python -c, the own_arg array and the set -u
    workaround it forced are gone. own-repo and own-source stay — neither is in _resolved.

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015RsLA9yt5sthqAuLFjBHdm
@mcocdawc
mcocdawc merged commit 83c6ffd into main Sep 10, 2026
30 checks passed
@mcocdawc
mcocdawc deleted the dep-table-takes-resolved branch September 10, 2026 14:14
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