GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Fix deprecated PyO3 code input import in benchmarks by youknowdot · Pull Request #9161 · RustPython/RustPython

Conversation

@youknowdot

@youknowdot youknowdot commented Oct 16, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Summary

Use pyo3::types::code::PyCodeInput in the execution benchmark. PyO3 0.30 deprecated the root types alias, which fails main's all-target Clippy jobs under -D warnings.

No lint suppression or behavior change.

Validation

  • Normal repository pre-commit hooks passed (including rustfmt and cspell).
  • Confirmed the canonical path against PyO3 v0.30.1 source.
  • Passed full workspace/all-target Clippy with -D warnings (excluding the separately configured C-API crate, wasm, and Windows venvlauncher), resolving PyO3 0.30.1.
  • Workspace and C-API unit tests are not yet completed locally. Exact-head GitHub CI is running and will be tracked before this is considered ready.

AI disclosure

OpenAI Codex assisted with diagnosis, the one-line patch, validation, and this draft PR. Model version unavailable.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
chatgpt-codex-connector[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@youknowdot
Fix deprecated PyO3 code input import in benchmarks
bf076f1 
Use the canonical code module path introduced in PyO3 0.30 without suppressing deprecation warnings.

Assisted-by: OpenAI Codex:model version unavailable
@coderabbitai

coderabbitai Bot commented Oct 16, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Repository: RustPython/RustPython/.coderabbit.yml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2ca31d00-3b67-454b-b262-5826914987bb
📥 Commits

Reviewing files that changed from the base of the PR and between fd5b30e and bf076f1.

📒 Files selected for processing: 1
  • benches/execution.rs
+--------------------------------------------------------------+
| Your function has more side effects than my caffeine intake. |
+--------------------------------------------------------------+
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
  • X
  • Mastodon
  • Reddit
  • LinkedIn

Comment @coderabbitai help to get the list of available commands.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
This was referenced Oct 16, 2026
Preserve function metadata and annotation callback lifetimes #9162
Merged
Implement basic REPL execution and fix piped stdin handling #9163
Merged
@fanninpm

fanninpm commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

cc @bschoenmaeckers
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@fanninpm
fanninpm requested a review from youknowone October 16, 2026 14:03
@fanninpm fanninpm mentioned this pull request Oct 16, 2026
build(deps): bump sha2 from 0.11.0 to 0.11.1 in the crypto group across 1 directory #9164
Merged
bschoenmaeckers
bschoenmaeckers approved these changes Oct 16, 2026
View reviewed changes
youknowone
youknowone approved these changes Oct 16, 2026
View reviewed changes
@youknowone
youknowone marked this pull request as ready for review October 16, 2026 14:43
@youknowone
youknowone merged commit b2b865a into RustPython:main Oct 16, 2026
21 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T14:44:58.705450Z bf076f1 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@youknowone youknowone youknowone approved these changes
+1 more reviewer
@bschoenmaeckers bschoenmaeckers bschoenmaeckers approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@youknowdot @fanninpm @youknowone @bschoenmaeckers
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.