Skip to content

Move prompt-format methods to main_modes/repl.py#1821

Merged
rolandwalker merged 1 commit intomainfrom
RW/move-prompt-string-methods-to-repl-py
Apr 5, 2026
Merged

Move prompt-format methods to main_modes/repl.py#1821
rolandwalker merged 1 commit intomainfrom
RW/move-prompt-string-methods-to-repl-py

Conversation

@rolandwalker
Copy link
Copy Markdown
Contributor

Description

Move these methods to main_modes/repl.py:

  • set_all_external_titles()
  • set_external_terminal_tab_title()
  • set_external_terminal_window_title()
  • set_external_multiplex_window_title()
  • set_external_multiplex_pane_title()
  • get_custom_toolbar()
  • get_prompt()

and other rearrangements needed to effect that change.

After the changes, main.py still has two calls to the new functions, which are marked with todo comments regarding the incomplete separation.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Apr 4, 2026
Move these methods to main_modes/repl.py:

 * set_all_external_titles()
 * set_external_terminal_tab_title()
 * set_external_terminal_window_title()
 * set_external_multiplex_window_title()
 * set_external_multiplex_pane_title()
 * get_custom_toolbar()
 * get_prompt()

and other rearrangements needed to effect that change.

After the changes, main.py still has two calls to the new functions,
which are marked with todo comments regarding the incomplete separation.
@rolandwalker rolandwalker force-pushed the RW/move-prompt-string-methods-to-repl-py branch from a4122e8 to e7c606f Compare April 4, 2026 23:55
@rolandwalker rolandwalker merged commit bec53b9 into main Apr 5, 2026
13 of 14 checks passed
@rolandwalker rolandwalker deleted the RW/move-prompt-string-methods-to-repl-py branch April 5, 2026 00:01
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.

1 participant