Skip to content

fix: skip unconfigured OpenAPI publishing - #8

Merged
DenisCarriere merged 1 commit into
masterfrom
fix/optional-openapi-publish
Aug 3, 2026
Merged

fix: skip unconfigured OpenAPI publishing#8
DenisCarriere merged 1 commit into
masterfrom
fix/optional-openapi-publish

Conversation

@DenisCarriere

@DenisCarriere DenisCarriere commented Aug 3, 2026

Copy link
Copy Markdown

Summary

  • skip the OpenAPI publish job when API_SPECS_REPOSITORY is not configured
  • keep OpenAPI generation and artifact validation running normally
  • preserve the existing publish behavior for repositories that configure a specs target and PAT

Why

The Pinax fork does not configure API_SPECS_REPOSITORY or API_SPECS_PAT. The inherited workflow generated every spec successfully, then failed solely because it always entered the optional cross-repository publish job.

Validation

  • mix format --check-formatted with Elixir 1.19.4 / OTP 27
  • git diff --check
  • actionlint syntax validation; the only unsuppressed findings were two pre-existing ShellCheck warnings outside this change
  • exact Generate OpenAPI Specs workflow: passed
  • all 18 OpenAPI variants generated; only the unconfigured publish job was skipped
  • default PR matrix and CodeQL: passed
  • one pre-existing market-history cache flake passed on rerun and is tracked in Stabilize recurring expanded CI test flakes #9

@DenisCarriere
DenisCarriere merged commit a455562 into master Aug 3, 2026
55 of 56 checks passed
@DenisCarriere
DenisCarriere deleted the fix/optional-openapi-publish branch August 3, 2026 14:52
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