Skip to content

fix: require compatible Snowflake CLI releases - #933

Merged
tdstein merged 1 commit into
mainfrom
symphony/posit-dev_rsconnect-python_930-448d70ab74f3b2cc
Nov 6, 2026
Merged

tdstein merged 1 commit into
mainfrom
symphony/posit-dev_rsconnect-python_930-448d70ab74f3b2cc

Conversation

@tdstein

@tdstein tdstein commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

Prevent unlocked installs from falling back to Snowflake CLI 3.7.1 and pydantic-core without Python 3.14 wheels.

- snowflake-cli
+ snowflake-cli>=2.8.2  # Python <3.10
+ snowflake-cli>=3.23.0 # Python >=3.10

Add a Python 3.14 CI job that installs the extra without the lock and checks the installed dependencies and CLI entry points.

Evidence

  • Before: unlocked Python 3.14 resolution selected Snowflake CLI 3.7.1 and pydantic-core 2.33.1.
  • After: a fresh Python 3.14 install selects CLI 3.32.1 and passes uv pip check; the minimum 3.23.0 also installs and passes. Both use wheels for pydantic-core, the connector and Snowpark; PyYAML builds from source.
  • Python 3.8 and 3.10 installs and dependency checks pass; unlocked resolution also checked for 3.9, 3.11 and 3.13.
  • 167 focused tests passed with Click 8.1.8; 12 skipped. Both projects require Click >=8.0, and posit-cli starts with the extra installed.
  • uv lock --locked and git diff --check pass.

Merge Danger

Door: two-way

Blast Radius: Snowflake extra consumers; Python 3.10+ users must resolve CLI 3.23.0 or newer.

Closes #930

@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-11-06 19:54 UTC

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
11113 9854 89% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 0c63cc7 by action🐍

@tdstein
tdstein enabled auto-merge (squash) November 6, 2026 19:48
@tdstein
tdstein requested a review from joshyam-k November 6, 2026 19:49
@tdstein
tdstein merged commit e6267ea into main Nov 6, 2026
29 checks passed
@tdstein
tdstein deleted the symphony/posit-dev_rsconnect-python_930-448d70ab74f3b2cc branch November 6, 2026 19:53
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.

snowflake extra resolves to snowflake-cli 3.7.1 and fails to install on Python 3.14

2 participants