feat: add opt-in live Overpass tests, LiveTests workflow, query fixes - #8
Merged
Merged
Conversation
Template SHA: 9ec6e46
…ws with template - _quarto.yml: point at docs/assets/* and docs/pages/*, adopt template theme (flatly + theme.scss, navbar logo), project-level engines: ['julia'] - Remove superseded docs/api.qmd, coverage.qmd, styles.css, _version-selector.html - docs/.gitignore: ignore _site/, pages/coverage/, pages/lcov.info - Docs.yml: template version (tag-push trigger, version/base-path injection, versions.json-based stable placeholder) with local precompile and render-retry kept - DocsBackfill.yml: template version (pages/coverage.qmd placeholder, version injection) - CLAUDE.md: drop stale open-cards note; API page now uses collapsible cards
- query: merge [out:json] and bbox into an existing settings statement, send a URL-encoded form body, and identify the client via User-Agent - test/test_service.jl: live queries against the public service, enabled by OVERPASS_LIVE_TESTS=true, 30 s retry on HTTP 429/504, mirror via OVERPASS_ENDPOINT - LiveTests workflow runs the live tests on a single ubuntu job; the main CI matrix stays offline - Drop :count from the query docstring; the parser has no case for count elements Claude-Session: https://claude.ai/code/session_01CfmYePXVFLWqxJisNJq39A
JuliaPackageTemplate's TagBot.yml declares no workflow-level permissions; TagBot relies on the repository default token permissions instead. Claude-Session: https://claude.ai/code/session_01CfmYePXVFLWqxJisNJq39A
joshday
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
querymerges[out:json]and thebboxkeyword into an existing settings statement instead of prepending a second one, sends a URL-encoded form body (Overpass rejects multipart with HTTP 400), and identifies the client with aUser-Agentheader as the public instance's usage policy asks.test/test_service.jlruns live queries against the public Overpass API: inline bbox,bboxkeyword with and without an existing settings block,OQLstatements, ways and relations without geom, and the HTTP 400 path for an invalid query. It retries HTTP 429/504 after 30 s and 60 s.OVERPASS_ENDPOINTselects a mirror.OVERPASS_LIVE_TESTS=true. The main CI matrix stays offline; the newLiveTestsworkflow runs them on a singleubuntu-latestjob with a per-ref concurrency group.query(::QLStatement)docstring no longer listsout=:count, which the parser does not handle.TagBot.ymldrops its workflow-levelpermissionsblock to match JuliaPackageTemplate.overpass-api.dewas refusing connections from the development IP after an earlier request burst. TheLiveTestsrun on this PR covers the default endpoint.mainin template-sync: apply JuliaPackageTemplate updates (TEMPLATE_SHA=9ec6e46) #7. They appear in the PR diff against the merge base but add nothing on merge, andgit merge-treereports no conflicts.https://claude.ai/code/session_01CfmYePXVFLWqxJisNJq39A