gh-131178: Add tests for `site` command-line interface #133582

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

ZeroIntensity merged 15 commits into python:main from ggqlq:add_tests_for_site_cli

Aug 15, 2025

+104 −0

Contributor

ggqlq commented May 7, 2025 •

edited by bedevere-app bot

Loading

Issue: Add tests for command-line interface #131178

ggqlq added 3 commits

May 7, 2025 22:35


          pythongh-131178: Add tests for command-line interface

fc36305


          pythongh-131178: Add tests for site command-line interface


          pythongh-131178: Add tests for site command-line interface

b1ec14d

ggqlq requested a review from FFY00 as a code owner

May 7, 2025 14:41

bedevere-app bot added the tests label

bedevere-app bot mentioned this pull request

Add tests for command-line interface #131178

Open

9 tasks

This comment was marked as resolved.

Sign in to view

bedevere-app bot added the awaiting review label

ggqlq marked this pull request as draft

May 7, 2025 16:09

bedevere-app bot removed the awaiting review label

This comment was marked as resolved.

Sign in to view

4 similar comments

This comment was marked as resolved.

Sign in to view

This comment was marked as resolved.

Sign in to view

This comment was marked as resolved.

Sign in to view

This comment was marked as resolved.

Sign in to view

ggqlq force-pushed the add_tests_for_site_cli branch from bd612e2 to b1ec14d Compare

May 7, 2025 19:00

This comment was marked as resolved.

Sign in to view


          Merge branch 'main' into add_tests_for_site_cli

1091b8d

This comment was marked as resolved.

Sign in to view

fix

6e8d75e

This comment was marked as resolved.

Sign in to view

ggqlq added 2 commits

May 8, 2025 13:44


          fix windows

3c1ba46


          fix windows

98145f6

This comment was marked as resolved.

Sign in to view

ggqlq marked this pull request as ready for review

May 8, 2025 06:51

bedevere-app bot added the awaiting review label

ZeroIntensity added the skip news label

ZeroIntensity reviewed

View reviewed changes

Lib/test/test_site.py Outdated Show resolved Hide resolved

Lib/test/test_site.py Outdated Show resolved Hide resolved

Lib/test/test_site.py Outdated Show resolved Hide resolved

Lib/test/test_site.py Outdated Show resolved Hide resolved

Member

ZeroIntensity commented May 11, 2025

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


          update

d36397b


          Merge branch 'main' into add_tests_for_site_cli

570fbf4

ggqlq requested a review from ZeroIntensity

May 13, 2025 06:59

ggqlq added 4 commits

May 25, 2025 10:15


          Merge branch 'main' into add_tests_for_site_cli

9bc46bc


          Merge branch 'main' into add_tests_for_site_cli

24bdf3f


          Merge branch 'main' into add_tests_for_site_cli

1a8f291


          Merge branch 'main' into add_tests_for_site_cli

d0999d2

ZeroIntensity added the 🔨 test-with-buildbots label

bedevere-bot commented Jun 5, 2025

🤖 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 removed the 🔨 test-with-buildbots label


          Merge branch 'main' into add_tests_for_site_cli

e9824d8

donBarbos mentioned this pull request

gh-131178: Add tests for site command-line interface #137807

Open


          Merge branch 'main' into add_tests_for_site_cli

d5fb77c

ZeroIntensity approved these changes

View reviewed changes

Member

ZeroIntensity left a comment

This looks good, thanks!

bedevere-app bot added awaiting merge and removed awaiting review labels

ZeroIntensity added needs backport to 3.13 needs backport to 3.14 labels

ZeroIntensity merged commit 03f5519 into python:main

50 checks passed

miss-islington-app bot commented Aug 15, 2025

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

bedevere-app bot removed the awaiting merge label

github-project-automation bot moved this to Done in lavitaconnect@MOSTAFAAMMER

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request


          pythongh-131178: Add tests for site command-line interface (pythonG…

2efeb8f

…H-133582)

(cherry picked from commit 03f5519)

Co-authored-by: ggqlq <124190229+ggqlq@users.noreply.github.com>

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request


          pythongh-131178: Add tests for site command-line interface (pythonG…

797f64e

…H-133582)

(cherry picked from commit 03f5519)

Co-authored-by: ggqlq <124190229+ggqlq@users.noreply.github.com>

bedevere-app bot commented Aug 15, 2025

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

bedevere-app bot removed the needs backport to 3.14 label

bedevere-app bot commented Aug 15, 2025

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

bedevere-app bot removed the needs backport to 3.13 label

ZeroIntensity pushed a commit that referenced this pull request


          [3.13] gh-131178: Add tests for site command-line interface (GH-133582

8d6e107

) (GH-137833)

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

Co-authored-by: ggqlq <124190229+ggqlq@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests