MobilityData / gtfs-validator

Canonical GTFS Validator project for schedule (static) files.
https://gtfs-validator.mobilitydata.org/
Apache License 2.0
288 stars 101 forks source link

feed_info.txt should be added to missing_required_file when translations.txt exists #1791

Closed mobilitydataio closed 2 months ago

mobilitydataio commented 4 months ago

Commit Nº 2

AUTHOR × DATE

https://github.com/google/transit/commit/cc60dc4275cc44662163efa8cefd3e9e5e504978

isabelle-dr commented 4 months ago

@emmambd heads-up that there is also #470, in this issue, where we've added:

The shapes.txt file should be included for all route-based services (not for zone-based demand-responsive services).

To validate this automatically, we'd need to do: if the feed contains services other than zone-based DRT services, shapes.txt should be included. cc @tzujenchanmbd

emmambd commented 4 months ago

@isabelle-dr Ah good catch! it didn't show up in the linked diff so I missed it :)

Alessandro100 commented 3 months ago

Tasks [] require feed_info.txt when translations.txt exists. Add conditional require to feed_info.txt [] test it with feed id (example feed with translations.txt and without feed_info.txt is here) [] review acceptance tests (if needed)

qcdyx commented 3 months ago

For clarification: The logic should be:

  1. when translations.txt exists, if there's no feed_info.txt, create missing required file notice.
  2. when there's no translation.txt, and no feed_info.txt either, create missing recommended file notice. Correct? @emmambd
emmambd commented 3 months ago

@qcdyx You got it! We're modifying feed_info.txt to be conditionally required in cases where translations.txt exists