Add E2E-based breaking change detection via CI#128
Add E2E-based breaking change detection via CI#128Aaravanand00 wants to merge 1 commit intohyperledger-identus:mainfrom
Conversation
b1ac1ef to
ccffd3e
Compare
There was a problem hiding this comment.
Pull request overview
Adds an additional CI signal intended to catch breaking changes earlier by running the repository’s existing E2E integration test entrypoint (npm run integration) on PRs and pushes to main.
Changes:
- Added a new GitHub Actions workflow to run E2E integration tests in CI.
- Updated README with a brief note about CI-based E2E breaking change detection.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| README.md | Documents that E2E tests run in CI for breaking-change detection. |
| .github/workflows/e2e-breaking-check.yml | Introduces a PR/push workflow intended to execute npm run integration -- --runner sdk-ts. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
43994da to
8e64e7f
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Aaravanand00 <aaravanand5749@gmail.com>
8e64e7f to
efd286c
Compare
|



Summary
This PR introduces a simple breaking change detection mechanism by running the existing E2E integration tests in CI.
What’s changed
npm run integrationcommand as the source of truth for E2E validationWhy this approach
Instead of introducing additional tooling, this relies on the current integration test setup as a practical signal for breaking changes, which keeps the solution simple and aligned with how the project is already tested.
Notes
sdk-tsrunner, aligned with the existing CI setup for Linux environments