Skip to content

feat(release): add targeted service scope#46

Merged
pedrolupin merged 1 commit into
mainfrom
codex/2242-chart-release-scope
Jul 14, 2026
Merged

feat(release): add targeted service scope#46
pedrolupin merged 1 commit into
mainfrom
codex/2242-chart-release-scope

Conversation

@pedrolupin

@pedrolupin pedrolupin commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an optional only input to chart release and force exactly the requested chart names
  • make Docker release accept the same comma-separated selector, including all parity
  • fail closed on invalid, blank, or unknown chart scopes
  • document identical Docker/chart selector wiring for service-repository callers

This is the shared-workflow prerequisite for pinpredict/trading#2242. The trading caller change will pass its services input to both release workflows after this contract lands.

Verification

  • go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.7 -color
  • exact chart prepare-step dry run: nadex-rfqgw selected only that chart
  • exact chart prepare-step dry run: two-chart CSV selected only those charts
  • invalid, unknown, and blank chart scopes failed closed
  • all selected all 14 trading charts without publishing
  • exact Docker args-step dry run: nadex-rfqgw produced only --only nadex-rfqgw
  • exact Docker args-step dry run: all resolved to all 14 trading image ids

@pedrolupin
pedrolupin force-pushed the codex/2242-chart-release-scope branch from 619a85a to b60c0c2 Compare July 14, 2026 15:51
@pedrolupin pedrolupin changed the title feat(chart-release): add targeted chart scope feat(release): add targeted service scope Jul 14, 2026
@pedrolupin
pedrolupin force-pushed the codex/2242-chart-release-scope branch from b60c0c2 to 5eba2fc Compare July 14, 2026 15:56
@pedrolupin
pedrolupin merged commit 9a41f07 into main Jul 14, 2026
2 checks passed
@pedrolupin
pedrolupin deleted the codex/2242-chart-release-scope branch July 14, 2026 16:59
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