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

Remove extra / in path #3167

Merged
merged 2 commits into from
Feb 12, 2023
Merged

Remove extra / in path #3167

merged 2 commits into from
Feb 12, 2023

Conversation

Castavo
Copy link
Contributor

@Castavo Castavo commented Feb 11, 2023

No description provided.

@Castavo Castavo requested a review from a team as a code owner February 11, 2023 23:05
@codecov
Copy link

codecov bot commented Feb 11, 2023

Codecov Report

Merging #3167 (fefdd85) into dev (ca00e2f) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                dev    #3167      +/-   ##
============================================
+ Coverage     40.45%   40.47%   +0.02%     
- Complexity     1709     1711       +2     
============================================
  Files           629      629              
  Lines         18698    18701       +3     
  Branches       2333     2333              
============================================
+ Hits           7564     7569       +5     
+ Misses        10731    10730       -1     
+ Partials        403      402       -1     
Flag Coverage Δ
core 83.96% <100.00%> (+0.04%) ⬆️

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

Impacted Files Coverage Δ
...e/src/main/java/fr/sncf/osrd/api/InfraManager.java 73.46% <ø> (ø)
core/src/main/java/fr/sncf/osrd/api/APIClient.java 66.66% <100.00%> (+5.12%) ⬆️
.../fr/sncf/osrd/api/ElectricalProfileSetManager.java 84.44% <100.00%> (+0.35%) ⬆️
...d/infra_state/implementation/TrainPathBuilder.java 83.33% <0.00%> (+1.38%) ⬆️

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

@multun multun enabled auto-merge (rebase) February 12, 2023 00:03
@multun multun merged commit c9b5254 into dev Feb 12, 2023
@multun multun deleted the bpt/remove-ep-sets-adress-slash branch February 12, 2023 00:06
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