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

Fix #335 Check that delay is not set for freq type 0 trips #337

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

scrudden
Copy link

Summary:
This implements the check detailed in issue #335.

It checks if a frequency "type 0" trip has a delay set and if so raises an E054 error.

Expected behavior:

It checks if any of the arrival and departure elements within stop_time_updates have a delay value set. If yes it raises an E054 error.

@scrudden scrudden changed the title Fix #335 Fix #335 Check that delay is not set for freq type 0 trips Sep 16, 2018
@barbeau
Copy link
Member

barbeau commented Sep 18, 2018

@scrudden Thanks for working on this too! This branch includes the E053 rule implemented in #336 too, so I'd say let's get #336 merged before reviewing this PR. Otherwise much of it will end up being affected by the changes/merge of #336.

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.

2 participants