-
Notifications
You must be signed in to change notification settings - Fork 120
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
added traffic tendecy #1479
added traffic tendecy #1479
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1479 +/- ##
=========================================
Coverage 76.53% 76.53%
Complexity 908 908
=========================================
Files 128 128
Lines 3929 3929
Branches 568 568
=========================================
Hits 3007 3007
Misses 675 675
Partials 247 247
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@VysotskiVadim could you add source code documentation as well (not javadoc)?
@@ -30,6 +30,6 @@ dependencies { | |||
|
|||
buildConfig { | |||
packageName = 'com.mapbox.sample' | |||
buildConfigField 'String', 'MAPBOX_ACCESS_TOKEN', System.getenv("MAPBOX_ACCESS_TOKEN") | |||
buildConfigField 'String', 'MAPBOX_ACCESS_TOKEN', project.hasProperty('MAPBOX_ACCESS_TOKEN') ? project.property('MAPBOX_ACCESS_TOKEN') : System.getenv('MAPBOX_ACCESS_TOKEN') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
TRAFFIC_TENDENCY_RAPIDLY_INCREASING_CONGESTION, | ||
TRAFFIC_TENDENCY_RAPIDLY_DECREASING_CONGESTION | ||
}) | ||
public @interface TrafficTendencyCriteria { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the annotation used anywhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only in comments
This reverts commit 43bae13.
6079ba3
to
a35f833
Compare
Added. I only had doubts if I should mention new annotation in route options docs. I decided not to mention and not to add comments there |
This is a private API, so absence of documentation is expected.
This feature was already introduced in the past but then reverted: #1451