Update to address optional location id - Githubissues

conveyal / gtfs-lib

A library for loading and saving GTFS feeds of arbitrary size with disk-backed storage

BSD 2-Clause "Simplified" License

71 stars 38 forks source link

Update to address optional location id #362

Closed br648 closed 2 years ago

br648 commented 2 years ago

Updated the packing of location groups to handled missing location id

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description

Location groups can optionally contain a location id. This updates handles the case where the location id has not been provided.

br648 commented 2 years ago

@miles-grant-ibigroup not sure if this actually required a PR, but hey.