Repository navigation
fix: repair the broken taxon filtering link fragment in the table config reference - #231
Merged
Merged
Conversation
SkyeAv
added this pull request to stack #235
October 21, 2026 21:09
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
SkyeAv
changed the title
…fig reference The taxon_from row linked to #taxon-filtering, but the prose section is titled Taxonomic Filtering, so the fragment never resolved; markdownlint's MD051 gate fails every pull request because CI runs only on pull_request. Point the link at
SkyeAv
force-pushed
the
fix/docs-taxon-filtering-link
branch
from
October 21, 2026 21:43
82e4b7a to
2564860
Compare
SkyeAv
added a commit
that referenced
this pull request
Oct 22, 2026
Cut 22.2.0 and bump the package version in pyproject.toml, CITATION.cff, and uv.lock. Minor: the --qc study stage fails the build on self-referential edges via a new self-loop-edges assertion (#229); eq/ne reindex filters can match cells with a Polars regex behind a default-off regex key (#232, #233, #234); source.reindex can filter rows by identifier membership with join/antijoin semi/anti hash joins (#236). Docs: the Taxon Filtering link in the table config reference now matches its section (#228, #231). Changelog: - Versioned the Unreleased section as 22.2.0 - 2026-10-22. - Added PR links to the three Added entries (#229; #232, #233, #234; #236). - Added the missing Fixed entry for the Taxon Filtering docs link (#228, #231). Docs: none needed here; the shipped docs changes are covered by the changelog entries above. Testing: - make check -> ruff, ruff format, pyright, pytest, cargo test, cargo clippy -D warnings: all green.
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.
fix: repair the broken taxon filtering link fragment in the table config reference
The
taxon_fromrow in the table config reference linked to#taxon-filtering, but the prose section is titledTaxonomic Filtering, so the fragment never resolved.markdownlint's MD051 gate fails every pull request because the markdown job runs only onpull_request, somainnever caught it.Fixes
docs/configuration/table.mdnow points at#taxonomic-filtering.Testing
npx -y markdownlint-cli2@0.23.3->0 issues in 0 files(was1 issue, MD051 at thetaxon_fromrow).Questions for the reviewer
docs/table-taxon-anchorfixes the same anchor; keep whichever lands first and drop the other.