Refactor CLI entry point to use main() function#17
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the CLI entry point to improve maintainability and programmatic access by introducing a main() function wrapper around the existing app() call. This follows Python packaging best practices for CLI applications.
- Introduces a new
main()function that callsapp() - Updates the
if __name__ == "__main__"block to callmain()instead ofapp()directly
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
|
|
||
| if __name__ == "__main__": | ||
| def main(): |
There was a problem hiding this comment.
The main() function should include a docstring to document its purpose as the CLI entry point, especially since it's intended for programmatic use from other modules.
Suggested change
| def main(): | |
| def main(): | |
| """ | |
| CLI entry point for fessctl. | |
| This function invokes the Typer application and can be called programmatically | |
| from other modules to start the CLI. | |
| """ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This update refactors the CLI entry point by introducing a main() function that calls app(). The if name == "main" block now invokes main() instead of calling app() directly. This change improves maintainability and allows the CLI to be executed programmatically from other modules without relying on the main check.