-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
integration-tests: deprecate #5926
Conversation
The legacy integration tests are no longer required. Delete the directory. This commit is helpful because it will reduce the amount of tests that need to run in CI. The legacy integration tests ran against the `ops` docker compose setup, which brought up the legacy system.
|
✅ Deploy Preview for opstack-docs canceled.
|
After deleting the `integration-tests` package
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #5926 +/- ##
===========================================
- Coverage 42.95% 42.63% -0.33%
===========================================
Files 478 422 -56
Lines 30783 29886 -897
Branches 877 679 -198
===========================================
- Hits 13224 12742 -482
+ Misses 16533 16163 -370
+ Partials 1026 981 -45
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bedrock ftw
Removes the seemingly obsolete rule to make integration tests from the root Makefile. Appears that integration tests were deprecated by PR ethereum-optimism#5926 and this rule no longer works.
Removes the seemingly obsolete rule to make integration tests from the root Makefile. Appears that integration tests were deprecated by PR ethereum-optimism#5926 and this rule no longer works.
Removes the seemingly obsolete rule to make integration tests from the root Makefile. Appears that integration tests were deprecated by PR ethereum-optimism#5926 and this rule no longer works.
Removes the seemingly obsolete rule to make integration tests from the root Makefile. Appears that integration tests were deprecated by PR #5926 and this rule no longer works.
Description
The legacy integration tests are no longer required. Delete the directory. This commit is helpful because it will reduce the amount of tests that need to run in CI.
The legacy integration tests ran against the
ops
docker compose setup, which brought up the legacy system.