Move json parsing to cli
Open
- Dominant language
- Python
- Stars
- 132
- Forks
- 21
- PR merge metrics
- No merged PRs in 30d
Description
Right now json parsing of tile `[{x}, {y}, {z}]` coordinates is hidden inside the api: https://github.com/mapbox/supermercado/blob/fc6513c79738ca32ad3ebbf653c9b88b7cd3773b/supermercado/uniontiles.py#L11
We should move this to the CLI and add `inputtiles` as a sequence / iterable.
Contributor guide
Research direction
Start with supermercado/uniontiles.py at the linked JSON-parsing code and trace how the CLI currently passes tile coordinates into the API. Move parsing of [{x}, {y}, {z}] coordinates to the CLI, make inputtiles a sequence or iterable, and verify that the API no longer owns this parsing responsibility.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api, cli
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100