-
Notifications
You must be signed in to change notification settings - Fork 38
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
Bugfix release #172
Bugfix release #172
Conversation
Take some reasonable changes that came out of #144 and add them here
at first I thought this was the first calendar date from the calendar_dates.txt file, but then I realized it was actually a result of calculating actual service of various days. Renaming this variable should reflect this distinction and avoid future confusion.
Fix duplicated schedule exceptions issue with multimap
Update GraphQL a little bit
Rename variable to avoid confusion
Conflicts: src/test/resources/snapshots/com/conveyal/gtfs/graphql/GTFSGraphQLTest/canFetchErrors.json
Handle zero hops separately if all travel times in feed are rounded
Codecov Report
@@ Coverage Diff @@
## master #172 +/- ##
============================================
+ Coverage 55.32% 55.56% +0.23%
- Complexity 737 749 +12
============================================
Files 143 143
Lines 7190 7311 +121
Branches 830 862 +32
============================================
+ Hits 3978 4062 +84
- Misses 2874 2899 +25
- Partials 338 350 +12
Continue to review full report at Codecov.
|
🎉 This PR is included in version 4.1.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Create a new bugfix release.
Fixes: