Skip to content

fix: migrate preview slug outputs to environment files - #138

Open
arunimshukla wants to merge 1 commit into
Consensys:mainfrom
arunimshukla:fix/github-output-slugs
Open

arunimshukla wants to merge 1 commit into
Consensys:mainfrom
arunimshukla:fix/github-output-slugs

Conversation

@arunimshukla

Copy link
Copy Markdown

Problem

Three docs preview composite actions still use the deprecated ::set-output workflow command to publish the generated PR slug. GitHub recommends writing step outputs to $GITHUB_OUTPUT.

Changes

  • Update docs-pages-deploy.
  • Update docs-pages-pr-deploy.
  • Update docs-pages-pr-delete.
  • Preserve the slug output name and every downstream reference.

Validation

  • Confirmed all three deprecated commands are removed.
  • Confirmed each step writes slug=PR-<number> to $GITHUB_OUTPUT.
  • Confirmed existing consumers still use steps.generate_pr_slug.outputs.slug.
  • Each action file changes one line only.

This branch has not been deployed

No deployments
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