Add missing transit modes in GTFS GraphQL InputModeWeight

[miklcct]

Summary

Add missing transit modes in GTFS GraphQL InputModeWeight

Issue

Fixes #6100

Unit tests

Not testable

Documentation

• GraphQL documentation added

[optionsome]

I think this addition is ok but we are starting to move away from using the old plan query, it's being deprecated in favour of the planConnection query in the scope of #6110. One small thing, the autogenerated data fetcher/type files automatically get unused/duplicate imports. Can you clean those away? If you are using intellij, it has an optimize imports action that does it.

[miklcct]

The file says do not edit by hand. Should we just leave them there as is instead? Otherwise it will confuse future contributors if the same change is generated over and over again. 4 Oct 2024 13:55:15 Joel Lappalainen ***@***.***>:

…

I think this addition is ok but we are starting to move away from using the old plan query, it's being deprecated in favour of the *planConnection* query in the scope of #6110[#6110]. One small thing, the autogenerated data fetcher/type files automatically get unused/duplicate imports. Can you clean those away? If you are using intellij, it has an optimize imports action that does it. — Reply to this email directly, view it on GitHub[#6101 (comment)], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAVSIAGWJVP6O4AB6O24W6DZZ2F3HAVCNFSM6AAAAABPDHZG76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJTGY2DKMRZHA]. You are receiving this because you authored the thread. [Tracking image][https://github.com/notifications/beacon/AAVSIADPKSEV34RGQQFHVRTZZ2F3HA5CNFSM6AAAAABPDHZG76WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUOVQQPE.gif]

[optionsome]

It's fine to clean up the imports there even though you shouldn't edit it by hand otherwise. We have discussed a while back should we clean those imports or not since they get generated every time but it's quite quick to do once you get used to it.

[habrahamsson-skanetrafiken]

This has merge conflicts, could you please resolve them?

[leonardehrenfried]

This needs a rebase. There was some confusing breakage when Github Actions installs pip packages. I hope it's now fixed on dev-2.x.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.

Project coverage is 69.91%. Comparing base (440498f) to head (0660e75).
Report is 60 commits behind head on dev-2.x.

Files with missing lines	Patch %	Lines
...ntripplanner/apis/gtfs/generated/GraphQLTypes.java	0.00%	20 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             dev-2.x    #6101      +/-   ##
=============================================
- Coverage      69.93%   69.91%   -0.02%     
+ Complexity     17730    17728       -2     
=============================================
  Files           1996     1996              
  Lines          75402    75422      +20     
  Branches        7717     7717              
=============================================
- Hits           52731    52730       -1     
- Misses         19995    20016      +21     
  Partials        2676     2676              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.