Skip to content

Decouples Astro CLI installation from the deploy action#136

Closed
odaneau-astro wants to merge 2 commits intoastronomer:mainfrom
odaneau-astro:main
Closed

Decouples Astro CLI installation from the deploy action#136
odaneau-astro wants to merge 2 commits intoastronomer:mainfrom
odaneau-astro:main

Conversation

@odaneau-astro
Copy link
Copy Markdown
Contributor

@odaneau-astro odaneau-astro commented Mar 17, 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

@odaneau-astro odaneau-astro requested a review from a team as a code owner March 17, 2026 22:21
@odaneau-astro
Copy link
Copy Markdown
Contributor Author

odaneau-astro commented Mar 18, 2026

Closing this PR to reopen a new one from a odaneau-astro: instead of odaneau-astro:main. See #137

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