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

core: stdcm: move java module from api to osrd #2958

Merged
merged 1 commit into from
Jan 20, 2023
Merged

Conversation

eckter
Copy link
Contributor

@eckter eckter commented Jan 20, 2023

This moves files around, nothing else. It was weird that all STDCM related files were in the api module. The endpoint and request/response files stay there and the rest is moved to a new stdcm module at the root of the project.

@eckter eckter requested a review from a team as a code owner January 20, 2023 13:17
@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #2958 (45f2eef) into dev (53b6a18) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #2958   +/-   ##
=========================================
  Coverage     38.29%   38.29%           
  Complexity     1713     1713           
=========================================
  Files           600      600           
  Lines         18342    18342           
  Branches       2513     2513           
=========================================
  Hits           7024     7024           
  Misses        10912    10912           
  Partials        406      406           
Flag Coverage Δ
core 83.88% <ø> (ø)

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

Impacted Files Coverage Δ
...ain/java/fr/sncf/osrd/api/stdcm/STDCMEndpoint.java 76.92% <ø> (ø)
...a/fr/sncf/osrd/stdcm/BacktrackingEnvelopeAttr.java 100.00% <ø> (ø)
...c/main/java/fr/sncf/osrd/stdcm/OccupancyBlock.java 100.00% <ø> (ø)
.../src/main/java/fr/sncf/osrd/stdcm/STDCMResult.java 100.00% <ø> (ø)
...va/fr/sncf/osrd/stdcm/UnavailableSpaceBuilder.java 94.87% <ø> (ø)
...ava/fr/sncf/osrd/stdcm/graph/AllowanceManager.java 91.89% <ø> (ø)
.../fr/sncf/osrd/stdcm/graph/BacktrackingManager.java 96.77% <ø> (ø)
...in/java/fr/sncf/osrd/stdcm/graph/DelayManager.java 100.00% <ø> (ø)
.../main/java/fr/sncf/osrd/stdcm/graph/STDCMEdge.java 76.47% <ø> (ø)
...ava/fr/sncf/osrd/stdcm/graph/STDCMEdgeBuilder.java 98.68% <ø> (ø)
... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@eckter eckter merged commit 4444f58 into dev Jan 20, 2023
@eckter eckter deleted the ech/move-stdcm-module branch January 20, 2023 13:50
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.

2 participants