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

Add Bikeep bicycle parking updater #5798

Merged
merged 3 commits into from
Apr 26, 2024

Conversation

leonardehrenfried
Copy link
Member

Summary

Adds a parking updater for the Estonian company Bikeep. This is a pure sandbox feature and only needs a single reviewer.

Unit tests

Added.

Documentation

Added.

cc @tsobuskerudbyen

Copy link

codecov bot commented Apr 10, 2024

Codecov Report

Attention: Patch coverage is 82.45614% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 67.90%. Comparing base (bad0b72) to head (f2a76c2).
Report is 21 commits behind head on dev-2.x.

❗ Current head f2a76c2 differs from pull request most recent head 091e4ed. Consider uploading reports for the commit 091e4ed to get more accurate results

Files Patch % Lines
...anner/ext/vehicleparking/bikeep/BikeepUpdater.java 84.09% 7 Missing ⚠️
...vehicleparking/bikeep/BikeepUpdaterParameters.java 50.00% 1 Missing ⚠️
...ner/ext/vehicleparking/parkapi/ParkAPIUpdater.java 50.00% 1 Missing ⚠️
...hicle_parking/VehicleParkingDataSourceFactory.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #5798      +/-   ##
=============================================
+ Coverage      67.89%   67.90%   +0.01%     
- Complexity     16556    16563       +7     
=============================================
  Files           1908     1910       +2     
  Lines          72341    72398      +57     
  Branches        7440     7440              
=============================================
+ Hits           49117    49165      +48     
- Misses         20702    20712      +10     
+ Partials        2522     2521       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

optionsome
optionsome previously approved these changes Apr 18, 2024
| Config Parameter | Type | Summary | Req./Opt. | Default Value | Since |
|----------------------------------|:---------------:|----------------------------------------------------------------------------|:----------:|---------------|:-----:|
| type = "vehicle-parking" | `enum` | The type of the updater. | *Required* | | 1.5 |
| [feedId](#u__13__feedId) | `string` | The name of the data source. | *Required* | | 2.2 |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This "name of the data source" is repeated in parking updater docs, maybe call it id instead of name.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated it in 091e4ed

Do you think that is a better way to explain it?

@t2gran t2gran added this to the 2.6 (next release) milestone Apr 25, 2024
@leonardehrenfried leonardehrenfried merged commit 0859b64 into opentripplanner:dev-2.x Apr 26, 2024
5 checks passed
@leonardehrenfried leonardehrenfried deleted the bikeep branch April 26, 2024 16:49
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.

3 participants