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

Bump io.mobilitydata.transit:gtfs-realtime-bindings, use imported gtfs-realtime.proto #22

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

Conversation

kurtraschke
Copy link
Contributor

@kurtraschke kurtraschke commented Nov 10, 2020

This is dependent on MobilityData/gtfs-realtime-bindings#70, and shouldn't be merged until that has been merged and a release made (which we presume will be called 0.0.7).

This PR updates io.mobilitydata.transit:gtfs-realtime-bindings to the latest release (0.0.7), and consequently removes the included copy of gtfs-realtime.proto in favor of the one included in the upstream JAR. It also adds JCenter directly as a Maven repository so we don't have to wait for gtfs-realtime-bindings releases to sync to Maven Central.

…0.0.7 release; consequently remove included copy of gtfs-realtime.proto and its exclusion in protoc configuration.

Add JCenter directly as a Maven repository so we don't have to wait for gtfs-realtime-bindings releases to sync to Maven Central.
@barbeau
Copy link
Member

barbeau commented Nov 12, 2020

0.0.7 of io.mobilitydata.transit:gtfs-realtime-bindings is live on JCenter now, but not Maven Central yet (and there is a sync issue I need to figure out there).

@kurtraschke kurtraschke marked this pull request as ready for review November 13, 2020 00:48
@kurtraschke kurtraschke changed the title WIP: bump io.mobilitydata.transit:gtfs-realtime-bindings Bump io.mobilitydata.transit:gtfs-realtime-bindings, use imported gtfs-realtime.proto Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants