Skip to content

Add support for transfers.txt to gtfs-schedule module#33

Merged
sargunv merged 3 commits intomainfrom
copilot/add-support-for-transfers-txt
Oct 23, 2025
Merged

Add support for transfers.txt to gtfs-schedule module#33
sargunv merged 3 commits intomainfrom
copilot/add-support-for-transfers-txt

Conversation

Copy link
Contributor

Copilot AI commented Oct 23, 2025 •
edited
Loading

  • Understand the existing codebase structure and patterns
  • Review GTFS specification for transfers.txt
  • Verify current tests pass
  • Create Transfer.kt model class with all fields following existing patterns
  • Create TransferTest.kt following existing test patterns
  • Uncomment transfer test lines in ProducerTest.kt
  • Run and verify all tests pass
  • Run linting and build verification
  • Format code and update ABI dump
  • Commit and push changes
  • Update minTransferTime to use WholeSeconds type instead of Int
Original prompt

Please add support for transfers.txt to the gtfs-schedule module. Use https://gtfs.org/documentation/schedule/reference/#transferstxt as a reference. Follow all existing patterns, especially the data types used and the tests. Remember to uncomment the relevant lines in producertest when you've added the model, or add it to the list at the bottom if none are available.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Oct 23, 2025
@sargunv
Add Transfer model and tests for transfers.txt support
ca534bc 
🤖 Generated with [GitHub Copilot](https://copilot.github.com)

Co-Authored-By: GitHub Copilot <noreply@github.com>

Co-authored-by: sargunv <1320357+sargunv@users.noreply.github.com>
sargunv
sargunv reviewed Oct 23, 2025
View reviewed changes
gtfs-schedule/src/commonMain/kotlin/dev/sargunv/mobilitydata/gtfs/schedule/Transfer.kt Outdated
* Amount of time, in seconds, that must be available to permit a transfer between routes at the
* specified stops.
*/
@SerialName("min_transfer_time") public val minTransferTime: Int? = null,
Copy link
Owner

@sargunv sargunv Oct 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot This should be the WholeSeconds type

Copy link
Contributor Author

Copilot AI Oct 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to use WholeSeconds type in debe497.

@sargunv sargunv marked this pull request as ready for review October 23, 2025 05:59
Copilot AI changed the title [WIP] Add support for transfers.txt in gtfs-schedule module Add support for transfers.txt to gtfs-schedule module Oct 23, 2025
Copilot AI requested a review from sargunv October 23, 2025 06:01
@sargunv
Use WholeSeconds type for minTransferTime field
debe497 
🤖 Generated with [GitHub Copilot](https://copilot.github.com)

Co-Authored-By: GitHub Copilot <noreply@github.com>

Co-authored-by: sargunv <1320357+sargunv@users.noreply.github.com>
@sargunv sargunv merged commit 4f3b5cc into main Oct 23, 2025
4 of 8 checks passed
@sargunv sargunv deleted the copilot/add-support-for-transfers-txt branch October 23, 2025 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@sargunv sargunv sargunv approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@sargunv sargunv

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sargunv