-
Notifications
You must be signed in to change notification settings - Fork 2
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
FE-187 remove copy project pipeline and tests #332
Conversation
… which I'm about to delete as well)
ef3675f
to
d863b1b
Compare
d863b1b
to
34b1f9c
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #332 +/- ##
=======================================
Coverage 95.56% 95.56%
=======================================
Files 4 4
Lines 158 158
Branches 5 5
=======================================
Hits 151 151
Misses 7 7 ☔ View full report in Codecov by Sentry. |
…ine-and-tests' into FE-187-remove-copy_project-pipeline-and-tests
…h stop the previously running jobs, if any.
…ine-and-tests' into FE-187-remove-copy_project-pipeline-and-tests
…ine-and-tests' into FE-187-remove-copy_project-pipeline-and-tests
…ine-and-tests' into FE-187-remove-copy_project-pipeline-and-tests
* skipping test_load_hca for now - FE-203 * linting
The e2e tests are failing (see FE-203) because they are running against TDR dev and TDR dev is apparently unstable. |
@@ -3,6 +3,9 @@ on: | |||
pull_request: | |||
branches: | |||
- main | |||
concurrency: | |||
group: ${{ github.ref }} | |||
cancel-in-progress: true |
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.
Interesting, is the idea that multiple e2e tests in flight are putting too much load on dev TDR?
Why
FE-187
This PR
Removes the copy_project pipeline, solids, and related tests. These were all unused and may have been causing lag during e2e testing.
Checklist