Skip to content

docs: Document single-PR deployment limitation and production workaround#622

Draft
Copilot wants to merge 2 commits intoci-fixfrom
copilot/sub-pr-617
Draft

docs: Document single-PR deployment limitation and production workaround#622
Copilot wants to merge 2 commits intoci-fixfrom
copilot/sub-pr-617

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

The README documented staging aggregation strategy but didn't clarify whether individual PRs can be deployed to staging in isolation for testing.

Changes

  • Staging Deployment section: Added note that staging always aggregates all open PRs, no single-PR isolation available
  • Manual Single-PR Deployment section: New section documenting:
    • deploy.yaml supports workflow_dispatch with pr_number input
    • Deploys to production (forrt.org), not staging
    • Recommended only for emergency fixes; use aggregated staging or local testing for regular PR review

Key Clarification

# deploy.yaml supports manual PR deployment but targets production
on:
  workflow_dispatch:
    inputs:
      pr_number:
        description: 'PR number to deploy (optional, for manual testing)'
        # Deploys to gh-pages (production), not staging

No single-PR staging deployment exists. Staging always shows combined state of all compatible open PRs.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: LukasWallrich <60155545+LukasWallrich@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix staging deployments to prevent duplicate builds docs: Document single-PR deployment limitation and production workaround Feb 5, 2026
Copilot AI requested a review from LukasWallrich February 5, 2026 12:17
@LukasWallrich
Copy link
Contributor

@richarddushime can we change the individual PR deploy to go to staging? Or does copilot get the documentation wrong?

@richarddushime
Copy link
Contributor

@richarddushime can we change the individual PR deploy to go to staging? Or does copilot get the documentation wrong?

the documentation by copilot is right , No single-PR staging deployment exists.Staging will always shows combined state of all compatible open PRs.,

Allowing a single PR deployment is what we are avoiding because the rest of the PRs changes wont be previewed on stagging site.
I also added a queue incase multiple PR s are running workflows to put on wait
and this will allow all PRs to deploy in order

Copy link
Contributor

@richarddushime richarddushime left a comment

Choose a reason for hiding this comment

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

This Documentation details would be nice if goes under contributing guidelines not in the main Readme file

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.

3 participants