Skip to content

Conversation

@philiporlando
Copy link
Contributor

@philiporlando philiporlando commented Oct 27, 2025

This spawned from #223 and documents that GTFS downloads requiring authentication are not supported by {tidytransit}.

  • Adds note to README.md
  • Adds note to {roxygen2} comment for read_gtfs() function
  • devtools::document() was executed to update the corresponding read_gtfs.Rd file.

@mpadge
Copy link
Contributor

mpadge commented Oct 28, 2025

@philiporlando I think having this in the README is a step too far. Happy for @polettif to decide, but feeds hidden behind paywalls or whatever is against the entire spirit of GTFS, as clearly indicated in the official overview:

Making a Transit feed publicly available

The simplest way to make a feed public is to host it on a web server and publish an announcement that makes it available for use.

GTFS should by definition be public. I think just removing the note from the README and leaving it as a side note in the function documentation only is fine.

@polettif
Copy link
Contributor

@philiporlando I think having this in the README is a step too far. Happy for @polettif to decide, but feeds hidden behind paywalls or whatever is against the entire spirit of GTFS, as clearly indicated in the official overview:

Yes, I agree. @philiporlando Thank you for the PR, please remove the note in the README then this is ready to get merged 👍

@philiporlando
Copy link
Contributor Author

philiporlando commented Oct 28, 2025

@philiporlando I think having this in the README is a step too far. Happy for @polettif to decide, but feeds hidden behind paywalls or whatever is against the entire spirit of GTFS, as clearly indicated in the official overview:

Yes, I agree. @philiporlando Thank you for the PR, please remove the note in the README then this is ready to get merged 👍

@polettif Done!

@polettif polettif merged commit b48743e into r-transit:master Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants