Skip to content

Fix deploy-pages workflow artifact step id conflict#4

Merged
megahertz merged 2 commits intomasterfrom
copilot/fix-deploy-script
Jan 15, 2026
Merged

Fix deploy-pages workflow artifact step id conflict#4
megahertz merged 2 commits intomasterfrom
copilot/fix-deploy-script

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 15, 2026

The deploy workflow failed due to conflicting id: deployment assignments between the upload and deploy steps.

Changes

  • Removed id: deployment from upload-pages-artifact step in build job
  • The deploy-pages step in deploy job retains id: deployment for page_url output reference

The workflow now follows the standard GitHub Pages deployment pattern where only the deploy-pages action needs an identifier.

  - uses: actions/upload-pages-artifact@v4
-   id: deployment
    with:
      path: packages/website/build
Original prompt

This section details on the original issue you should resolve

<issue_title>Fix website deploy script</issue_title>
<issue_description></issue_description>

<agent_instructions>Fix deploy script

Action log: https://github.com/megahertz/react-dialogues/actions/runs/21044337968/job/60514507403</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: megahertz <1762672+megahertz@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix website deploy script Fix deploy-pages workflow artifact step id conflict Jan 15, 2026
Copilot AI requested a review from megahertz January 15, 2026 20:01
@megahertz megahertz marked this pull request as ready for review January 15, 2026 20:04
@megahertz megahertz merged commit 281bdc2 into master Jan 15, 2026
2 checks passed
@megahertz megahertz deleted the copilot/fix-deploy-script branch January 15, 2026 20:14
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.

Fix website deploy script

2 participants