Skip to content

[CLI] Include tools in Lighthouse package - #265

Merged
adam-smnk merged 1 commit into
llvm:mainfrom
adam-smnk:package-cli-tools
Aug 21, 2026
Merged

[CLI] Include tools in Lighthouse package#265
adam-smnk merged 1 commit into
llvm:mainfrom
adam-smnk:package-cli-tools

Conversation

@adam-smnk

Copy link
Copy Markdown
Member

Includes core Lighthouse tools (opt, run, tune) in the python package.

After installing lighthouse, the three lh-* commands are now available out of the box.

Includes core Lighthouse tools (opt, run, tune) in the python package.

After installing lighthouse, the three lh-* commands are now available
out of the box.
@adam-smnk
adam-smnk requested a review from rengolin August 21, 2026 10:45

@rengolin rengolin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: why change the Py file to snake case?

@adam-smnk

Copy link
Copy Markdown
Member Author

nit: why change the Py file to snake case?

It's the standard convention for .py module naming as per PEP8 and all other Python files in repo.

@adam-smnk
adam-smnk merged commit c2787c9 into llvm:main Aug 21, 2026
3 of 5 checks passed
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.

2 participants