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

Extract journey request from RouteRequest #4446

Merged
merged 9 commits into from
Sep 15, 2022

Conversation

hannesj
Copy link
Contributor

@hannesj hannesj commented Sep 12, 2022

Summary

This is a refactor of the RoutingRequest

Issue

Contiues from #4426
closes #4156

@hannesj hannesj added this to the 2.2 milestone Sep 12, 2022
@hannesj hannesj requested a review from a team as a code owner September 12, 2022 13:19
t2gran
t2gran previously approved these changes Sep 14, 2022
Bartosz-Kruba
Bartosz-Kruba previously approved these changes Sep 14, 2022
@codecov-commenter
Copy link

Codecov Report

Base: 58.43% // Head: 58.49% // Increases project coverage by +0.05% 🎉

Coverage data is based on head (c35043c) compared to base (0c9bf26).
Patch coverage: 63.45% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #4446      +/-   ##
=============================================
+ Coverage      58.43%   58.49%   +0.05%     
- Complexity     11386    11425      +39     
=============================================
  Files           1501     1508       +7     
  Lines          59814    59954     +140     
  Branches        6832     6831       -1     
=============================================
+ Hits           34954    35068     +114     
- Misses         22792    22827      +35     
+ Partials        2068     2059       -9     
Impacted Files Coverage Δ
...qlapi/datafetchers/LegacyGraphQLQueryTypeImpl.java 6.14% <0.00%> (-0.03%) ⬇️
...er/ext/transmodelapi/TransmodelGraphQLPlanner.java 0.00% <0.00%> (ø)
...tripplanner/ext/traveltime/TravelTimeResource.java 0.00% <0.00%> (ø)
...rg/opentripplanner/api/common/RoutingResource.java 0.00% <0.00%> (ø)
...va/org/opentripplanner/api/model/ApiItinerary.java 100.00% <ø> (ø)
.../opentripplanner/api/resource/PlannerResource.java 0.00% <ø> (ø)
...planner/graph_builder/module/NearbyStopFinder.java 84.76% <0.00%> (-1.65%) ⬇️
...java/org/opentripplanner/model/plan/Itinerary.java 76.15% <ø> (ø)
...va/org/opentripplanner/routing/RoutingService.java 26.66% <0.00%> (-0.92%) ⬇️
...outing/api/request/preference/BikePreferences.java 95.74% <ø> (ø)
... and 52 more

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@t2gran t2gran merged commit 8370272 into opentripplanner:dev-2.x Sep 15, 2022
@hannesj hannesj deleted the otp2_journey_request branch September 15, 2022 08:56
t2gran pushed a commit that referenced this pull request Sep 15, 2022
@t2gran
Copy link
Member

t2gran commented Sep 15, 2022

I merged this with one approval, since we had two approvals before @hannesj merged the dev-2.x into this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Part A - Via search, refactor OTP Internal programming API
4 participants