Add API-doc site, format gate, recipe.yaml, bench - #3
Merged
Conversation
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
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.
Suite-wide quick wins, following the pattern established in mojo-xml and mojo-feed:
mojo formatoutput as its own commit, then added a format gate to the test workflow (mojo format+git diff --exit-code) so CI fails if sources drift from canonical format.docs/render_api.py(same renderer as mojo-xml) plus adocs.yamlworkflow that buildsmojo docJSON into a static HTML API reference and deploys it to GitHub Pages.mojopin, not a compiled.mojopkg). Not wired to CI.urlparse/urlunparseround-trips andparse_qsl/urlencoderound-trips over the CPython byte-match fixture corpus (test/data/fixtures.txt). Corpus is tiny, so it iterates 20k times for stable numbers. Localpixi run benchonly, not in CI.fmt,docs,benchadded topixi.toml;.gitignoreextended for generated artifacts.mojo docemits no summary warning).Verified locally:
pixi run test(38/38 pass, 173/173 fixture byte-match),pixi run docs,pixi run bench, and the format-gate simulation (pixi run fmt && git diff --exit-code) all pass.🤖 Generated with Claude Code