Merged
Conversation
sjquant
reviewed
Apr 9, 2026
.github/workflows/polars-release.yml
Outdated
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-python@v5 | ||
| with: | ||
| python-version: "3.13" |
Collaborator
There was a problem hiding this comment.
전체적으로 python-version을 3.13으로 쓰고 있는거 같습니다.
지원 범위가 >=3.10이고 ABI 기준도 abi3-py310이면 단일 버전만 유지할 경우 최소 지원 버전인 3.10을 쓰는 편이 더 보수적일거 같습니다.
Contributor
Author
There was a problem hiding this comment.
찾아보니까, setup-python은 host runner의 python 버전을 지정하는데,
maturin-action은 호스트 내에 별도로 띄운 컨테이너에서 maturin build를 돌리나봐요. 그래서 setup-python의 python 버전이 영향을 미치지 않는다고 합니다! (참고)
Collaborator
There was a problem hiding this comment.
uv run pytest 등도 여향을 받기 때문에 python 3.10으로 잡는게 보수적일거 같습니다!
sjquant
reviewed
Apr 13, 2026
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.
작업 배경
polars-techr를 현재 레이아웃 그대로 유지하면서 실제 배포 가능한 상태로 정리했습니다.이번 PR의 목적은 다음 세 가지입니다.
주요 변경 사항
패키징/메타데이터
polars/pyproject.tomlpolars>=1.39,<2로 런타임 의존성 범위 조정dynamic = ["version"]로 변경해서 Python 패키지 버전을Cargo.toml에서 가져오도록 변경maturinexclude 설정 추가polars/Cargo.tomlabi3-py310으로 ABI 기준 정렬core/Cargo.toml검증 스크립트
polars/scripts/check_artifacts.py__pycache__,*.pyc같은 불필요 파일 포함 여부 검사polars/scripts/smoke_import.pyimport polars_techr및 기본 expression 호출 확인Cargo.toml하나로만 관리합니다.CI / 릴리스 워크플로우
.github/workflows/polars-ci.ymlreview_requested,ready_for_review시점에만 실행concurrency로 같은 ref의 중복 실행 취소.github/workflows/polars-release.ymlpolars-vX.Y.Z태그 시 PyPI 배포workflow_dispatch시 TestPyPI 배포개발 흐름 정리
Makefiletest-polars전에maturin develop --uv수행build-prod-polars가 wheel + sdist를 함께 생성하도록 조정polars/README.mdCargo.toml기준으로 정리리뷰 포인트
pyproject.toml의 dynamic version 구성이 현재 배포 정책과 맞는지polars-ci트리거를 review 시점으로 늦춘 정책이 팀 운영 방식과 맞는지검증
cargo test -p techr-corecd polars && uv run pytestcd polars && uv run maturin build --release --sdist --out distcd polars && uv run python scripts/check_artifacts.py distuv run --isolated --python 3.13 --with .../polars_techr-0.1.0-...whl python .../smoke_import.py후속 작업
이 PR이 머지되면
main기준으로Polars Releaseworkflow를 수동 실행해서 TestPyPI 업로드를 먼저 검증한 뒤, 이상 없으면polars-vX.Y.Z태그로 PyPI 정식 배포를 진행하면 됩니다.