Skip to content

chore: splits the workflows into 2#399

Open
Eomm wants to merge 6 commits intomainfrom
iso-work
Open

chore: splits the workflows into 2#399
Eomm wants to merge 6 commits intomainfrom
iso-work

Conversation

@Eomm
Copy link
Member

@Eomm Eomm commented Feb 3, 2026

Will write a description

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR splits the monolithic CI-CD workflow into two separate workflows to better separate concerns between preview deployments and production deployments.

Changes:

  • Created a new deploy-website.yml workflow dedicated to production deployments triggered by pushes to main/next/v* branches or manual/external triggers
  • Modified preview.yml to focus solely on pull request previews, removing production deployment logic and the setup job
  • Simplified both workflows by removing conditional logic and the setup job that was previously used to determine the target environment

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/preview.yml Streamlined to handle only pull request previews, removing production deployment job and setup job, now uses sandbox environment
.github/workflows/deploy-website.yml New workflow dedicated to production deployments, extracted from the original CI-CD workflow

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Manuel Spigolon <behemoth89@gmail.com>
permissions:
contents: read
environment:
name: sandbox
Copy link
Member Author

Choose a reason for hiding this comment

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

@Fdawgs this is the core change

Copy link
Member

Choose a reason for hiding this comment

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

Thanks @Eomm, this tackles the issues with pull_request_target does it?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, it does

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