Skip to content

Decouple Astro CLI installation from the deploy action#137

Open
odaneau-astro wants to merge 2 commits intoastronomer:mainfrom
odaneau-astro:setup-astro-cli-decoupling
Open

Decouple Astro CLI installation from the deploy action#137
odaneau-astro wants to merge 2 commits intoastronomer:mainfrom
odaneau-astro:setup-astro-cli-decoupling

Conversation

@odaneau-astro
Copy link
Copy Markdown
Contributor

@odaneau-astro odaneau-astro commented Mar 18, 2026

We will be introducing a new setup-astro-cli GitHub Action. This PR removes Astro CLI installation steps from the deploy action.

  • The cli-version input and the "Install Astro CLI" step have been removed.
  • The action now expects the CLI to already be available in PATH and will fail with a clear error pointing to astronomer/setup-astro-cli if it is not.
  • All README workflow examples have been updated to include an explicit astronomer/setup-astro-cli step.
  • Implement intermediate environment variables to handle untrusted inputs for deploy-type and wait-time. See https://docs.github.com/en/actions/reference/security/secure-use#use-an-intermediate-environment-variable

Copy link
Copy Markdown
Member

@feluelle feluelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's going to be a breaking change, which means that this can only be included in v1.

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.

2 participants