Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CLI Changes Sub-PR: New structure and parsing logic #591

Merged
merged 10 commits into from
Aug 14, 2023
Merged

CLI Changes Sub-PR: New structure and parsing logic #591

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
4c493d7
Add env feature to clap and cargo update
Xophmeister Jul 17, 2023
64c4cfc
Dead Code: WIP CLI changes
Xophmeister Jul 17, 2023
73e1ee6
Dead Code: CLI feature parity
Xophmeister Jul 19, 2023
53ff983
Note about not (yet) using infer_subcommands
Xophmeister Jul 19, 2023
8d7ad09
Use clap/wrap_help
Xophmeister Jul 19, 2023
405bc02
WIP: Strip out previous CLI argument parser setup
Xophmeister Jul 21, 2023
00339e9
WIP: Normalise arguments for caller
Xophmeister Jul 21, 2023
1d93f48
Don't check for file-ness in the CLI argument parser
Xophmeister Jul 24, 2023
a6a163b
File and directory canonicalisation for the argument parser
Xophmeister Jul 24, 2023
338af25
Expose CLI types so they can be used downstream
Xophmeister Aug 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
83 changes: 47 additions & 36 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion topiary-cli/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@ path = "src/main.rs"
[dependencies]
# For now we just load the tree-sitter language parsers statically.
# Eventually we will want to dynamically load them, like Helix does.
clap = { workspace = true, features = ["derive"] }
# NOTE clap/wrap_help isn't perfect (see clap-rs/clap#5022)
clap = { workspace = true, features = ["derive", "env", "wrap_help"] }
env_logger = { workspace = true }
futures = { workspace = true }
directories = { workspace = true }
Expand Down