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.

Sign up for GitHub

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

Jump to bottom

Derive the `session.id`/`order_number` from the `upload.id` #720

Draft

Swatinem wants to merge 3 commits into main from swatinem/fix-sessionid-allocation

Draft

Derive the `session.id`/`order_number` from the `upload.id` #720

fix various foreign key column definitions

Codecov Notifications / codecov/patch failed Sep 18, 2024 in 0s

67.16% of diff hit (target 98.07%)

View this Pull Request on Codecov

67.16% of diff hit (target 98.07%)

Annotations

Check warning on line 1193 in services/report/__init__.py

codecov-notifications / codecov/patch

services/report/__init__.py#L1193

Added line #L1193 was not covered by tests

Check warning on line 96 in services/report/raw_upload_processor.py

codecov-notifications / codecov/patch

services/report/raw_upload_processor.py#L96

Added line #L96 was not covered by tests

Check warning on line 39 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L36-L39

Added lines #L36 - L39 were not covered by tests

Check warning on line 44 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L44

Added line #L44 was not covered by tests

Check warning on line 196 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L195-L196

Added lines #L195 - L196 were not covered by tests

Check warning on line 198 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L198

Added line #L198 was not covered by tests

Check warning on line 200 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L200

Added line #L200 was not covered by tests

Check warning on line 212 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L212

Added line #L212 was not covered by tests

Check warning on line 215 in tasks/tests/integration/test_upload_e2e.py

codecov-notifications / codecov/patch

tasks/tests/integration/test_upload_e2e.py#L215

Added line #L215 was not covered by tests

Check warning on line 135 in tasks/upload_finisher.py

codecov-notifications / codecov/patch

tasks/upload_finisher.py#L135

Added line #L135 was not covered by tests

Check warning on line 163 in tasks/upload_finisher.py

codecov-notifications / codecov/patch

tasks/upload_finisher.py#L163

Added line #L163 was not covered by tests

Check warning on line 542 in tasks/upload_finisher.py

codecov-notifications / codecov/patch

tasks/upload_finisher.py#L541-L542

Added lines #L541 - L542 were not covered by tests

Check warning on line 546 in tasks/upload_finisher.py

codecov-notifications / codecov/patch

tasks/upload_finisher.py#L546

Added line #L546 was not covered by tests

Check warning on line 558 in tasks/upload_finisher.py

codecov-notifications / codecov/patch

tasks/upload_finisher.py#L556-L558

Added lines #L556 - L558 were not covered by tests

Check warning on line 356 in tasks/upload_processor.py

codecov-notifications / codecov/patch

tasks/upload_processor.py#L356

Added line #L356 was not covered by tests

View more details on Codecov Notifications