-
Notifications
You must be signed in to change notification settings - Fork 3
Calebfinalpush #15
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.
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
Calebfinalpush #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds final user-friendly support, help, and getting started commands, updates command naming to use hyphens, refreshes documentation and metadata, and polishes the TUI and README for launch.
- Introduce
/getting-started,/support, and/discordcommands with custom TUI panels - Rename commands and flags to consistent hyphenation and update help formatter
- Update
pyproject.tomlmetadata and refresh README and ASCII art instructions
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Add keywords, update authors, classifiers, and script |
| chuck_data/ui/tui.py | Handle new support, examples, and Discord panels |
| chuck_data/ui/help_formatter.py | Hyphenate command names and adjust alias formatting |
| chuck_data/ui/ascii_art.py | Update welcome text and example prompts |
| chuck_data/commands/support.py | New support command handler |
| chuck_data/commands/getting_started.py | New getting-started command handler |
| chuck_data/commands/discord.py | New Discord command handler |
| chuck_data/commands/setup_stitch.py | Add informational messages before launch |
| chuck_data/commands/wizard/steps.py | Remove obsolete model reference |
| chuck_data/commands/wizard/renderer.py | Remove obsolete model reference |
| chuck_data/commands/init.py | Register new commands |
| README.md | Refresh intro, commands list, and installation examples |
Comments suppressed due to low confidence (2)
chuck_data/ui/tui.py:510
- Using cmd.startswith("/help") will also match '/help-me', causing the support block to be skipped. Consider matching exactly '/help' or moving this condition after more specific '/help-me' or '/support' checks.
elif (cmd.startswith("/help")
chuck_data/ui/help_formatter.py:161
- [nitpick] Flags here use underscores ('--catalog_name') but commands use hyphens elsewhere. Rename to '--catalog-name' and '--schema-name' for consistency.
alias = f"{alias} [--catalog_name <catalog>] [--schema_name <schema>]"
Final polish work to prep for launch. Lots in here mostly around creating friendly support commands, updating docs, etc.