search

codeforanchorage / realtime-bus-sms

Text back current arrival time for buses given a texted in bus stop number for People Mover. AddressLookup branch would allow texting "Arctic and Fireweed"
https://bus.codeforanchorage.org
14 stars 10 forks source link

GTFS auto - updates vs git repo #129

Closed mark-meyer closed 6 years ago

mark-meyer commented 6 years ago

Since we have started automatically updating GTFS, it has become a little inconvenient to deal with GTFS files in the git repo. Here's what now happens:

  1. Clone repo locally
  2. This includes the GTFS files, but they often old
  3. The app downloads new GTFS zip file and expands contents
  4. Local repo now shows changed files - to do anything like make a new branch, we need to either stash or commit these changes.

So at this point do we just commit the new files? This makes for some big commits with a lot of changes that we don't actually control.

Would it be better to add the gtfs directory to .gitignore?

tescher commented 6 years ago

I vote for .gitignore

Tim Escher tim@eschers.com 608-432-5799

On Thu, Jan 25, 2018 at 9:56 AM, Mark Meyer notifications@github.com wrote:

Since we have started automatically updating GTFS, it has become a little inconvenient to deal with GTFS files in the git repo. Here's what now happens:

  1. Clone repo locally
  2. This includes the GTFS files, but they often old
  3. The app downloads new GTFS zip file and expands contents
  4. Local repo now shows changed files - to do anything like make a new branch, we need to either stash or commit these changes.

So at this point do we just commit the new files? This makes for some big commits with a lot of changes that we don't actually control.

Would it be better to add the gtfs directory to .gitignore?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/codeforanchorage/realtime-bus-sms/issues/129, or mute the thread https://github.com/notifications/unsubscribe-auth/ACcF3ND1x-gj5qG8tpmRLnyc2gItPrJjks5tOM5ggaJpZM4RtUph .

mark-meyer commented 6 years ago

Changed in the last pull. GTFS is no longer in the repo.