Skip to content

Enable python_shared_standard_library compat flag by default - #8141

Merged
ryanking13 merged 1 commit into
mainfrom
gyeongjae/stdlib-default
Nov 3, 2026
Merged

ryanking13 merged 1 commit into
mainfrom
gyeongjae/stdlib-default

Conversation

@ryanking13

Copy link
Copy Markdown
Contributor

Turns on python_shared_standard_library compat flag by default. I added it in #7780 and tested locally.

@ryanking13
ryanking13 requested review from a team as code owners November 2, 2026 05:07

@clanky-the-clanker clanky-the-clanker Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Superseded by the review of 8a523adf.

This review, of 43779a83

✅ Approved — One-line compatibility-date change that follows the existing impliedByAfterDate pattern; no issues found.

The decision is advisory: this review is a comment and never approves, blocks or requests changes on the pull request.


✅ All checks passed

📋 Code Quality: ✅ LGTM

The change follows the existing $impliedByAfterDate(name = "pythonWorkers", ...) pattern. Opt-in via python_shared_standard_library still works.


Reviewed commit: 43779a83

@ryanking13
ryanking13 enabled auto-merge November 3, 2026 03:23
@ryanking13
ryanking13 force-pushed the gyeongjae/stdlib-default branch from 43779a8 to 8a523ad Compare November 3, 2026 03:47

@clanky-the-clanker clanky-the-clanker Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Approved — One-line compatibility-date change that follows the existing impliedByAfterDate pattern; no issues found.

The decision is advisory: this review is a comment and never approves, blocks or requests changes on the pull request.


✅ All checks passed

📋 Code Quality: ✅ LGTM

The change follows the existing $impliedByAfterDate(name = "pythonWorkers", ...) pattern. Opt-in via python_shared_standard_library still works.


Reviewed commit: 8a523adf — same changes as 43779a83, so this is that commit's review (no new review was run; line numbers are as of 43779a83)

@ryanking13
ryanking13 merged commit b690abb into main Nov 3, 2026
24 of 25 checks passed
@ryanking13
ryanking13 deleted the gyeongjae/stdlib-default branch November 3, 2026 04:56
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.

3 participants