Skip to content
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

editoast: add timetable import endpoint #4784

Merged
merged 1 commit into from
Sep 5, 2023

Conversation

flomonster
Copy link
Contributor

close #4579

@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Merging #4784 (0013e1a) into dev (453ce8a) will decrease coverage by 0.02%.
The diff coverage is 23.33%.

@@             Coverage Diff              @@
##                dev    #4784      +/-   ##
============================================
- Coverage     22.35%   22.33%   -0.02%     
  Complexity     2276     2276              
============================================
  Files           863      864       +1     
  Lines         90288    90422     +134     
  Branches       2324     2324              
============================================
+ Hits          20183    20195      +12     
- Misses        68828    68950     +122     
  Partials       1277     1277              
Flag Coverage Δ
editoast 69.48% <23.33%> (-1.24%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
editoast/src/core/simulation.rs 50.00% <ø> (ø)
...oast/src/models/infra_objects/operational_point.rs 0.00% <0.00%> (ø)
editoast/src/models/rolling_stock/mod.rs 83.63% <0.00%> (-12.20%) ⬇️
editoast/src/models/timetable.rs 76.74% <0.00%> (-11.26%) ⬇️
editoast/src/models/train_schedule.rs 34.61% <ø> (ø)
editoast/src/schema/rolling_stock/mod.rs 87.50% <ø> (ø)
editoast/src/views/mod.rs 95.12% <ø> (ø)
editoast/src/views/train_schedule/mod.rs 13.72% <0.00%> (+0.34%) ⬆️
editoast/src/views/timetable.rs 6.81% <2.72%> (-8.11%) ⬇️
editoast/src/views/pathfinding/mod.rs 91.00% <96.66%> (+0.47%) ⬆️
... and 3 more

... and 2 files with indirect coverage changes

@flomonster flomonster force-pushed the fam/add-import-timetable-endpoint branch 3 times, most recently from b7fc5fc to 408960b Compare August 10, 2023 16:46
@flomonster flomonster force-pushed the fam/add-import-timetable-endpoint branch 7 times, most recently from 4fa01b9 to a156905 Compare August 29, 2023 12:20
@flomonster flomonster force-pushed the fam/add-import-timetable-endpoint branch from a156905 to f9ea76c Compare August 30, 2023 14:40
@flomonster flomonster force-pushed the fam/add-import-timetable-endpoint branch from f9ea76c to 0013e1a Compare August 30, 2023 15:54
@flomonster flomonster marked this pull request as ready for review August 30, 2023 15:56
@flomonster flomonster requested a review from a team as a code owner August 30, 2023 15:56
Copy link
Contributor

@Tguisnet Tguisnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM

@flomonster flomonster added this pull request to the merge queue Sep 4, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 4, 2023
@flomonster flomonster merged commit 451d16d into dev Sep 5, 2023
7 of 8 checks passed
@flomonster flomonster deleted the fam/add-import-timetable-endpoint branch September 5, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

back : Add a timetable import endpoint /timetable/{id}/import
2 participants