Skip to content

Commit 0c82b3f

Browse files
chore: update to maturin's recommended project layout for rust/python… (#695)
* chore: update to maturin's recommended project layout for rust/python projects The previous layout leads to an import error when installing with `maturin build` and `pip install .`. This error was common enough that `maturin` changed the recommended project layout to what this commit does. A prior PR attempted to solve this by altering `lib.name` in Cargo.toml, but that did not work for me. - [Prior PR](#694) - [maturin ImportError issue](PyO3/maturin#490) - [maturin changes recommended project structure](PyO3/maturin#855) * ci: update `ruff check` for nested python directory
1 parent 01a370e commit 0c82b3f

29 files changed

+2
-1
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
pip install ruff
3939
# Update output format to enable automatic inline annotations.
4040
- name: Run Ruff
41-
run: ruff check --output-format=github datafusion
41+
run: ruff check --output-format=github python/
4242

4343
generate-license:
4444
runs-on: ubuntu-latest

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ repository = "https://github.com/apache/arrow-datafusion-python"
5555
profile = "black"
5656

5757
[tool.maturin]
58+
python-source = "python"
5859
module-name = "datafusion._internal"
5960
include = [
6061
{ path = "Cargo.lock", format = "sdist" }

0 commit comments

Comments
 (0)