Skip to content

Conversation

@FilipPyrek
Copy link

@FilipPyrek FilipPyrek commented Jan 20, 2026

Summary by CodeRabbit

  • Chores

    • Updated default Python baseline from 3.9 to 3.10.
    • Added support for Python 3.11, 3.12 and 3.13.
    • Removed Python 3.9 support and related legacy image/build artifacts.
  • CI / Workflows

    • Adjusted workflows and build matrices to use 3.10 as the IR baseline and include the expanded Python matrix.
    • Increased CI step timeouts for several long-running build and push jobs.
  • Documentation

    • Updated supported tags and usage notes to reflect the new Python versions.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

This pull request removes Python 3.9 support and establishes Python 3.10 as the baseline across the project. CI (.circleci/config.yml) timeouts were increased for long-running steps and default Python versions/matrices updated to 3.10–3.13. Dockerfiles (ir/ir-base and python/conda) now default ARG PYTHON_VERSION to 3.10. The python3.9-specific Dockerfile was deleted. Documentation (python/readme.md) and supported tags were updated to reflect 3.10–3.13.

Sequence Diagram(s)

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: remove python 3.9' directly and clearly describes the primary change across all modified files—removing Python 3.9 support and updating to 3.10+.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@FilipPyrek FilipPyrek force-pushed the fpyrek/drop-python-3-9 branch from 7f0ca05 to a106892 Compare January 22, 2026 15:53
@deepnote deepnote deleted a comment from linear bot Jan 22, 2026
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@python/readme.md`:
- Around line 5-6: The Markdown heading "Supported Tags and Respective
Dockerfile Links" lacks a blank line after it; update the section by inserting a
single blank line immediately after the heading (i.e., after "# Supported Tags
and Respective Dockerfile Links") so the following list line with the tags
renders correctly and satisfies markdownlint.
- Around line 6-7: The referenced constraint files (URLs
https://tk.deepnote.com/constraints3.10.txt through constraints3.13.txt) are
missing and breaking builds; either deploy the missing constraint files at those
exact URLs or remove the --constraint references and README links that point to
them. Update python/readme.md to stop linking the nonexistent constraint URLs
and edit the Dockerfiles that reference these constraints (e.g., the Dockerfiles
in python/python and python/datascience/Dockerfile.datascience) to either (a)
remove the pip --constraint <url> flags or (b) replace them with valid hosted
constraint files; ensure the Dockerfile pip install lines and README entries
consistently reflect the chosen fix.

@FilipPyrek
Copy link
Author

FilipPyrek commented Jan 23, 2026

The check R with libs 4.4.3 has been running for 30 mins already and it's still building and logging. 🤔 and it times out

@FilipPyrek FilipPyrek force-pushed the fpyrek/drop-python-3-9 branch from a6c9b80 to 7db4bfb Compare January 23, 2026 16:22
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.

4 participants