Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature] Netex IDF: stop accessibility #460

Merged
merged 2 commits into from
Nov 15, 2019

Conversation

datanel
Copy link
Member

@datanel datanel commented Nov 14, 2019

No description provided.

woshilapin
woshilapin previously approved these changes Nov 14, 2019
@@ -66,7 +66,6 @@ where
}

collections.enhance_with_co2();
// TODO: uncomment once we have all netex parsed
// collections.sanitize()?;
collections.sanitize()?;

This comment was marked as resolved.

stop_points.push(stop_point)?;
}

Ok(stop_points)
let mut equipments: Vec<_> = equipments.into_iter().map(|(_, e)| e).collect();
equipments.sort_unstable_by(|tp1, tp2| tp1.id.cmp(&tp2.id));

This comment was marked as resolved.

@datanel datanel force-pushed the netex_stop_accessibility branch from a315c92 to 5e507c2 Compare November 15, 2019 09:28
@mergify mergify bot dismissed woshilapin’s stale review November 15, 2019 09:28

Pull request has been modified.

woshilapin
woshilapin previously approved these changes Nov 15, 2019
@datanel datanel force-pushed the netex_stop_accessibility branch from 5e507c2 to 71b4198 Compare November 15, 2019 10:01
@mergify mergify bot dismissed woshilapin’s stale review November 15, 2019 10:01

Pull request has been modified.

@datanel datanel force-pushed the netex_stop_accessibility branch from 71b4198 to f9f056f Compare November 15, 2019 10:36
@mergify mergify bot merged commit 70567c9 into hove-io:master Nov 15, 2019
@datanel datanel deleted the netex_stop_accessibility branch November 15, 2019 11:13
@datanel datanel changed the title Netex IDF: stop accessibility [feature] Netex IDF: stop accessibility Nov 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants