Skip to content

Conversation

@emersonknapp
Copy link
Member

@emersonknapp emersonknapp commented Oct 5, 2025

That way instead of

from launch_frontend_py import launch
from launch_frontend_py.actions import executable

A user only has to remember a single module import

from launch_frontend_py import launch, executable

Highlighted changes:

  • Collapsed the contents of action.py in to __init__.py to simplify the handling of __all__ for that toplevel module.
  • Add make_valid_name to handle the underscore suffixing, so it's done the same way both places it's used

Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
@emersonknapp emersonknapp merged commit d8f5284 into main Oct 5, 2025
5 checks passed
@emersonknapp emersonknapp deleted the all-one-module branch October 5, 2025 22:20
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