docs: fix misaligned row in README.rst pre-trained models table - #20
Merged
Conversation
The `hybrid_cv01_s5775.zip` row in the grid table was one space short in
its first cell, so its column borders sat at 1/43/54/61 instead of the
1/44/55/62 used by every other row. docutils rejects the whole table
("Malformed table. Right border not aligned or missing."), which made
`twine check --strict` fail on the README (the package long_description)
in the push-to-main build_package job.
Pad the row to 62 chars so the borders line up; `twine check --strict`
now passes on both the wheel and the sdist.
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.
Problem
The Push to main run after #19 merged fails in
build_packageat the Check build artifacts step (uvx twine check --strict dist/*):pyproject.tomlsetsreadme = "README.rst", soREADME.rstis the packagelong_description. The grid table under Pre-trained models has one misaligned row: thehybrid_cv01_s5775.ziprow was one space short in its first cell, putting its column borders at 1/43/54/61 instead of the 1/44/55/62 used by every other row and border line. docutils rejects the whole table, and--strictturns that into a failure.Cause / why it wasn't caught
86913ae, already onmainbefore fix: correct continuous-action PPO scaling, seeding, and eval defaults; retrain pinned models #19. The fix: correct continuous-action PPO scaling, seeding, and eval defaults; retrain pinned models #19 merge just re-ran the job that surfaces it.pull_request_to_main.ymlhas thebuild_packagejob commented out, sotwine checkonly runs post-merge (push_to_main.yml), never on PRs.Fix
One space added to the
hybrid_cv01_s5775.ziprow so it is 62 chars wide and its borders line up. No other changes.Verification
docutils at twine's report level — no "Malformed table":
Full CI-equivalent: