Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Fix #37 - Add best practice to include shapes.txt #38

Closed
wants to merge 1 commit into from

Conversation

e-lo
Copy link

@e-lo e-lo commented May 5, 2021

Summary:

Fixes #37 to add a best practice to include shapes.txt

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with npm test to make sure you didn't break anything NA
  • Format the title like "Fix #<issue_number> - " (for example - "Fix #1111 - Check for null value before using field")
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s) - NA

@e-lo e-lo mentioned this pull request Jul 26, 2021
@scmcca
Copy link

scmcca commented Dec 16, 2021

My thoughts at #37 (comment)

@Sergiodero
Copy link
Contributor

As GTFS Best Practices (BP) are currently in the process of being merged to the specification, MobilityData is migrating outstanding issues and PRs from this repository to google/transit. Thus, this PR will be closed and further discussion regarding this BP should take place in google/transit. Please refer to Issue #421 for a more detailed explanation of the migration process and the proposed next steps.

With this, we’re hoping to bring more visibility to outstanding BP issues and to restart the discussion around them, so that any improvement that the community finds valuable could be carried forward.

@Sergiodero Sergiodero closed this Dec 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposed best practice: shapes.txt should be included
3 participants