Deprecate v1.x / merge 2.x into master
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 25/100
Research direction
Start by reviewing the 2.x branch, master, and the linked MBGL migration context to determine whether the proposed merge is still appropriate. Then resolve the checklist decisions: merging 2.x, documenting v1.x deprecation and vtzero versus mapbox/vector-tile usage in the README, and considering the repository rename; done means the agreed plan is completed and documented.
Written by the indexing model from the issue text.
Description
Currently the only (known) application depending on the v1 version of this library is MBGL. Once MBGL moves to vtzero (https://github.com/mapbox/mapbox-gl-native/pull/11489) I propose we move forward with v2.x here by:
- merging v2.x branch (https://github.com/mapbox/vector-tile/tree/2.x) into master
- adding a deprecation note in the readme that using v1.x is no longer recommended for us
- adding a section of the readme that describes when you'd use vtzero vs mapbox/vector-tile
- consider renaming this repo to
mapbox/libvectortilesincemapbox/vector-tileis so easily confused given its generic name
Sound like a good plan? @flippmoke @kkaefer @anandthakker @joto
Context: v1.x of this library was never well tested. Vtzero is extremely well tested. The v2.x branch is now based on top of vtzero and is therefore a higher level, more robust replacement.
- Dominant language
- C++
- Stars
- 52
- Forks
- 33
- PR merge metrics
- No merged PRs in 30d
Getting set up
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from mapbox/vector-tile
-
1.x
Difficulty 4/5 3-5 days Newbie friendliness 25/100
mapbox/vector-tile#45 · 5 comments ·
-
2.x
Difficulty 5/5 Over a week Newbie friendliness 20/100
mapbox/vector-tile#43 ·
-
2.x
Difficulty 5/5 Over a week Newbie friendliness 30/100
mapbox/vector-tile#42 ·
-
Add OverzoomingOpen2.x
Difficulty 5/5 Over a week Newbie friendliness 25/100
mapbox/vector-tile#41 ·
-
Add Encoding from WGS 84 / Web MercMay be free again @flippmoke claimed this 3159 days ago, and no pull request is open. Open2.x
mapbox/vector-tile#40 · 1 assignee ·
All issues in mapbox/vector-tile
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
hyprwm/aquamarine#426 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
amnezia-vpn/amnezia-client#3222 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
sdatkinson/NeuralAmpModelerCore#342 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
valkey-io/valkey-search#1465 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
KhronosGroup/Vulkan-Tutorial#524 ·
Maintainers usually reply within 1 day