Skip to content

Conversation

@ggqlq
Copy link
Contributor

@ggqlq ggqlq commented May 7, 2025

@ggqlq ggqlq requested a review from FFY00 as a code owner May 7, 2025 14:41
@bedevere-app bedevere-app bot added the tests Tests in the Lib/test dir label May 7, 2025
@bedevere-app

This comment was marked as resolved.

@ggqlq ggqlq marked this pull request as draft May 7, 2025 16:09
@bedevere-app

This comment was marked as resolved.

4 similar comments
@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@ggqlq ggqlq force-pushed the add_tests_for_site_cli branch from bd612e2 to b1ec14d Compare May 7, 2025 19:00
@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@ggqlq ggqlq marked this pull request as ready for review May 8, 2025 06:51
@ZeroIntensity
Copy link
Member

Also: don't force push, it just makes reviewing harder and the bot louder. We squash-merge at the end.

@ZeroIntensity ZeroIntensity added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jun 5, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @ZeroIntensity for commit d0999d2 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F133582%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jun 5, 2025
Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

This looks good, thanks!

@ZeroIntensity ZeroIntensity added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Aug 15, 2025
@ZeroIntensity ZeroIntensity merged commit 03f5519 into python:main Aug 15, 2025
50 checks passed
@miss-islington-app
Copy link

Thanks @ggqlq for the PR, and @ZeroIntensity for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 15, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 15, 2025
@bedevere-app
Copy link

bedevere-app bot commented Aug 15, 2025

GH-137832 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Aug 15, 2025
@bedevere-app
Copy link

bedevere-app bot commented Aug 15, 2025

GH-137833 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Aug 15, 2025
ZeroIntensity pushed a commit that referenced this pull request Aug 15, 2025
) (GH-137833)

gh-131178: Add tests for `site` command-line interface  (GH-133582)
(cherry picked from commit 03f5519)

Co-authored-by: ggqlq <[email protected]>
@ggqlq
Copy link
Contributor Author

ggqlq commented Aug 18, 2025

@ZeroIntensity Thank you for your review!

Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
ZeroIntensity pushed a commit that referenced this pull request Oct 7, 2025
) (GH-137832)

gh-131178: Add tests for `site` command-line interface  (GH-133582)
(cherry picked from commit 03f5519)

Co-authored-by: ggqlq <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants