search

MobilityData / gtfs-flex

NOTICE: GTFS-Flex has been merged to GTFS. This repo is no longer up-to-date and will deprecated. Consult the google/transit repo for the up-to-date info.
https://github.com/google/transit
Apache License 2.0
119 stars 24 forks source link

Update reference.md and README.md to Flex v2 #42

Closed westontrillium closed 3 years ago

CLAassistant commented 4 years ago

CLA assistant check
All committers have signed the CLA.

LeoFrachet commented 4 years ago

The only tiny tiny comment I would make is that MobilityData standardized the naming of GTFS extensions, and that therefore we slightly tend to advocate to write "GTFS-Flex" instead of "GTFS-flex".

But this is tiny tiny, and thanks for putting together the proposal and assembling it. Great job!

I've shared a few thoughts in the email I sent you just a bit earlier @thomastrillium, but overall LGTM

tsherlockcraig commented 4 years ago

Much appreciated and I have actually been thinking about this recently whenever I type GTFS-flex and GTFS-Fares next to one another :)

Weston could you make this update?

On Fri, Oct 30, 2020 at 12:37 PM Leo Frachet notifications@github.com wrote:

The only tiny tiny comment I would make is that MobilityData standardized the naming of GTFS extensions, and that therefore we slightly tend to advocate to write "GTFS-Flex" instead of "GTFS-flex".

But this is tiny tiny, and thanks for putting together the proposal and assembling it. Great job!

I've shared a few thoughts in the email I sent you just a bit earlier @thomastrillium https://github.com/thomastrillium, but overall LGTM

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MobilityData/gtfs-flex/pull/42#issuecomment-719756769, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZ6ZNHOIBBH3ZURW43YPULSNMIW7ANCNFSM4S2OHVTQ .

westontrillium commented 4 years ago

@thomastrillium Capitalization change made to README.md.

tsherlockcraig commented 3 years ago

@scmcca I don't think I have the creds to merge this PR. Is that something you could go ahead and do?