Skip to content

gh-118908: Limit exposed globals from internal imports and definitions on new REPL startup #119547

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

Merged
pablogsal merged 19 commits into from
Jun 11, 2024
+83 −8
Merged

gh-118908: Limit exposed globals from internal imports and definitions on new REPL startup #119547

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
083f0a9
Limit exposed globals in new REPL
eugenetriguba May 25, 2024
bf17576
add blurb
eugenetriguba May 25, 2024
891477b
Merge branch 'main' into gh-118908
eugenetriguba May 25, 2024
2fabf57
tweak test
eugenetriguba May 25, 2024
d5d2321
fix type hint
eugenetriguba May 25, 2024
21a16f8
Add is_using_pyrepl
eugenetriguba May 26, 2024
5c5c496
Merge remote-tracking branch 'upstream/main' into gh-118908
pablogsal Jun 11, 2024
a9c0854
Simplify implementation
pablogsal Jun 11, 2024
d8760e8
Fix mypy
pablogsal Jun 11, 2024
bc49d05
Fix mypy
pablogsal Jun 11, 2024
adb074a
Fix windows
pablogsal Jun 11, 2024
d9a1085
Merge remote-tracking branch 'upstream/main' into gh-118908
pablogsal Jun 11, 2024
cdbc556
Fix generated files
pablogsal Jun 11, 2024
eaa6bc8
Fix generated files
pablogsal Jun 11, 2024
782b231
removed old file
pablogsal Jun 11, 2024
097f21e
Restore old file
pablogsal Jun 11, 2024
549a8d2
update news entry
pablogsal Jun 11, 2024
c95c035
fixup! update news entry
pablogsal Jun 11, 2024
688129b
Update Lib/test/test_pyrepl/test_pyrepl.py
lysnikolaou Jun 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix type hint
  • Loading branch information
@eugenetriguba
eugenetriguba committed May 25, 2024
commit d5d2321719073fa69dfd1983923b45632a46035a
2 changes: 1 addition & 1 deletion Lib/test/test_pyrepl/test_pyrepl.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -775,7 +775,7 @@ def test_dumb_terminal_exits_cleanly(self):
self.assertNotIn("Exception", output)
self.assertNotIn("Traceback", output)
Comment thread
pablogsal marked this conversation as resolved.

def run_repl(self, repl_input: str | list[str], env: dict | None = None) -> str:
def run_repl(self, repl_input: str | list[str], env: dict | None = None) -> tuple[str, int]:
master_fd, slave_fd = pty.openpty()
process = subprocess.Popen(
[sys.executable, "-i", "-u"],
Expand Down