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

Bug fix release #175

Merged
merged 33 commits into from
Jan 4, 2019
Merged

Bug fix release #175

merged 33 commits into from
Jan 4, 2019

Conversation

landonreed and others added 29 commits September 26, 2018 18:11
…r trips

This refactor includes a check for Pattern#useFrequency when either a pattern is updated or trips
are updated in order to appropriately trigger other behaviors (for example, keeping stop times in
sync with patter stops travel times or disallowing frequency entries for timetable-based patterns).

fixes #173
Conflicts:
	src/test/java/com/conveyal/gtfs/GTFSTest.java
Validate stop names and trip headsigns
Update trip#shape_id and frequency stop_times on pattern updates
…raveled

Validate stop_time#shape_dist_traveled
@landonreed landonreed requested a review from evansiroky January 4, 2019 16:47
@codecov-io
Copy link

codecov-io commented Jan 4, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@057a962). Click here to learn what that means.
The diff coverage is 63.63%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #175   +/-   ##
=========================================
  Coverage          ?   57.21%           
  Complexity        ?      791           
=========================================
  Files             ?      144           
  Lines             ?     7276           
  Branches          ?      845           
=========================================
  Hits              ?     4163           
  Misses            ?     2773           
  Partials          ?      340
Impacted Files Coverage Δ Complexity Δ
src/main/java/com/conveyal/gtfs/loader/Table.java 85.22% <ø> (ø) 75 <0> (?)
.../java/com/conveyal/gtfs/loader/JdbcGtfsLoader.java 70.33% <0%> (ø) 27 <0> (?)
src/main/java/com/conveyal/gtfs/loader/Feed.java 75% <0%> (ø) 6 <0> (?)
...rc/main/java/com/conveyal/gtfs/model/StopTime.java 70% <100%> (ø) 2 <0> (?)
...java/com/conveyal/gtfs/error/NewGTFSErrorType.java 100% <100%> (ø) 2 <0> (?)
.../java/com/conveyal/gtfs/loader/SnapshotResult.java 100% <100%> (ø) 1 <1> (?)
.../com/conveyal/gtfs/validator/ServiceValidator.java 89.59% <100%> (ø) 29 <0> (?)
...a/com/conveyal/gtfs/graphql/GraphQLGtfsSchema.java 98.25% <100%> (ø) 1 <0> (?)
.../java/com/conveyal/gtfs/error/SQLErrorStorage.java 72.22% <100%> (ø) 12 <1> (?)
src/main/java/com/conveyal/gtfs/GTFS.java 46.44% <100%> (ø) 10 <0> (?)
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 057a962...444b3ce. Read the comment docs.

Landon Reed and others added 4 commits January 4, 2019 11:50
Increase headway seconds max to 6 hours
this way it will upload a report for the code coverage after all the commits made with maven-semantic-release have completed.  Codecov looks at the most recent commit to master to compare the next report to, so this is needed for code coverage comparisons on PRs to master.
Run codecov after semantic-release
@evansiroky evansiroky assigned landonreed and unassigned evansiroky Jan 4, 2019
@landonreed landonreed merged commit 099f632 into master Jan 4, 2019
@landonreed
Copy link
Contributor Author

🎉 This PR is included in version 4.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants