Skip to content

sqlfluffrs: Add version 4.3.0 - #3095

Merged
riseproject-dev[bot] merged 2 commits into
mainfrom
sqlfluffrs
Oct 25, 2026
Merged

riseproject-dev[bot] merged 2 commits into
mainfrom
sqlfluffrs

Conversation

@luhenry

@luhenry luhenry commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Compiles SQLFluff's Rust lexer/parser extension (pyo3, abi3-py310) with maturin. Upstream publishes no riscv64 wheel.

Mirrors upstream's publish-sqlfluffrs-release-to-pypi.yaml, with the wheel test taken from the -rust jobs of ci-tests.yml.

Differs from upstream

  • Nothing beyond the riscv64 target and the manylinux_2_39 image.

Matrix: one cp310-abi3 wheel, tested on 3.12, 3.13 and 3.14. No free-threaded wheel, matching upstream.

Testing

  • Runs the tox py3XX-rust environment's not integration suite against the wheel. The linting, typing and docs tools from requirements_dev.txt are left out.

License: OK

luhenry added a commit that referenced this pull request Oct 13, 2026
All three test_wheel legs failed the same three cases of
test/core/config/loader_test.py::test__config__get_user_config_dir_path.
The failure is not riscv64-specific: sqlfluff 4.3.0's mock_xdg_home
fixture sets XDG_CONFIG_HOME to the unexpanded "~/.config/my/special/path",
and platformdirs 4.12.0 and later ignore a non-absolute XDG_CONFIG_HOME
as the XDG Base Directory spec requires, so the XDG path is never
consulted. It reproduces on x86_64 with platformdirs >= 4.12.0 and passes
with 4.11.0.

Upstream fixed the fixture in sqlfluff 4.4.0 (commit 2dd9c7c7, #8461).
Carry that commit as a test-only patch applied in the test_wheel job;
the build job and the wheel are unchanged.
@github-actions
github-actions Bot marked this pull request as ready for review October 25, 2026 17:50
@riseproject-dev
riseproject-dev Bot merged commit 9845c75 into main Oct 25, 2026
10 checks passed
@riseproject-dev
riseproject-dev Bot deleted the sqlfluffrs branch October 25, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant