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

Remove yaml CLI. #880

Closed
fmassot opened this issue Dec 10, 2021 · 1 comment
Closed

Remove yaml CLI. #880

fmassot opened this issue Dec 10, 2021 · 1 comment
Assignees
@ddelemeny
Labels
enhancement New feature or request
Projects
Quickwit 0.3

Comments

@fmassot
Copy link
Contributor

fmassot commented Dec 10, 2021

There are good reasons to remove it: clap-rs/clap#3087
@fmassot fmassot added the enhancement New feature or request label Dec 10, 2021
@fmassot fmassot added this to To do in Quickwit 0.2 Dec 11, 2021
@fulmicoton
Copy link
Contributor

fulmicoton commented Dec 14, 2021 •
edited

It does not have to be done in 0.2 I think though?
I'm removing it from the quickwit 0.2 project. (Please readd it if I am missing something)

@fulmicoton fulmicoton removed this from To do in Quickwit 0.2 Dec 14, 2021
ddelemeny added a commit to ddelemeny/quickwit that referenced this issue Feb 10, 2022
@ddelemeny
Build CLI without clap's load_yaml. See quickwit-oss#880
f4b2399
@fmassot fmassot added this to Done in Quickwit 0.3 Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ddelemeny ddelemeny

Labels
enhancement New feature or request
Projects
No open projects
Quickwit 0.3
Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fmassot @fulmicoton @ddelemeny