mapbox / mapbox/vector-tile-spec

Remove Optional From Layer Extent

Open
#47 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

3.0 Specification
Dominant language
No language data
Stars
1k
Forks
213
PR merge metrics
No merged PRs in 30d

Description

Currently within `message Layer`, the `extent` field is optional. In order to not change the format of the `.proto` in version 2 this has been only noted with a comment that is it required. In a later version of the release this should be changed to required.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the `message Layer` definition and inspect the `extent` field and its comment, along with the surrounding versioning context. Done means the field is changed from optional to required in a later release without changing the version 2 `.proto` format.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.