ci: split e2e to independent workflow#339
Conversation
There was a problem hiding this comment.
Pull request overview
This PR splits the E2E test job from the main CI workflow into an independent workflow file, allowing E2E tests to run separately from the main CI pipeline.
- Extracts the
e2e-testjob fromci.ymlinto a dedicatede2e-test.ymlworkflow - Configures the new workflow to run only on main branch pushes and manual dispatch
- Duplicates the
download-previous-rolldown-binariesjob in the new workflow to maintain job dependencies
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/e2e-test.yml | New independent workflow file containing the E2E test job and its dependency on rolldown binaries download |
| .github/workflows/ci.yml | Removes the e2e-test job that was previously conditional on main branch |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Brooooooklyn
force-pushed
the
12-04-ci-split-e2e
branch
from
6fd1f0b to
9b7d801
Compare
Brooooooklyn
force-pushed
the
12-04-ci-split-e2e
branch
from
9b7d801 to
e6348f5
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
No description provided.