Repository navigation
Add a Navigation help group and fix stale docs counts - #91
Conversation
zmofei
added
the
locationai-team-skills
The renderer matched the service name tilequery, which #116 merged into tilesets, so tilesets query has printed pretty JSON since 0.2.0. Match tilesets, and check every listed name against the bundled specs so a future rename fails a test instead of dropping the list.
Fix stale counts in docs and restore the tilesets query list
mattpodwysocki
left a comment
mattpodwysocki
left a comment
There was a problem hiding this comment.
Confirmed this is what's currently blocking main: directions/isochrone/map-match/matrix landed after #83 added the help-group tests, so every_command_has_a_place, every_api_command_has_a_description and top_level_help_groups_commands_and_options are red on main right now. Rebuilt this branch and verified directly: the real --help output matches the Navigation block in the description exactly, the 39/13 operation and group counts in docs/commands.md check out against a real --schema run, and the tilequery->tilesets rename fix (with every_listed_service_is_a_real_command_group) is in here too. Full suite, fmt and clippy all pass.
This is also the fix PR #82 got stacked on, so merging this brings that one's content into main as well.
Approving, and merging now since this is unblocking CI on main for everyone.
zmofei
changed the title
3 participants
CI on main fails because
directions,isochrone,map-matchandmatrixhave no help group or description since #83. This adds a Navigation group for them.#82 was merged into this branch and lands with it. The last commit fixes what it left in
docs/commands.md: the missingstyles downloadentry, the commands that answer with bytes, and one oldtilequeryname.Verified: fmt, clippy, tests; counts checked against
mapbox --schema. Not run against the live API.