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.

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

Document command line arguments #75

Open
lobre opened this issue Nov 30, 2023 · 9 comments
Open

Document command line arguments #75

lobre opened this issue Nov 30, 2023 · 9 comments

Comments

@lobre
Copy link
Contributor

lobre commented Nov 30, 2023

I see that you parse 3 command line arguments in the main logic of vi: -s, -e and -v.

https://github.com/aligrudi/neatvi/blob/master/vi.c#L1653

I am struggling to understand what they do exactly, and I can't find this info in the README either.

Do you mind explaining their meaning? And if so, does it make sense to document that in the README?

@lobre
Copy link
Contributor Author

lobre commented Dec 1, 2023

I saw you just added them with -h. Thank you!

Maybe I can suggest also to mention that a file can be provided.

vi [options] [file]

And as neatvi supports multiple buffers, maybe having it take multiple files as input could also be nice.

vi [options] [file ...]

I did not know that neatvi has ex mode. That is great.

@aligrudi
Copy link
Owner

aligrudi commented Dec 1, 2023 via email

@lobre
Copy link
Contributor Author

lobre commented Dec 12, 2023

As Neatvi does not support next and prev commands, I just added the first form.

Would this make sense now that neatvi supports next and prev? (b+/b-)

@aligrudi
Copy link
Owner

aligrudi commented Dec 14, 2023 via email

@aligrudi
Copy link
Owner

aligrudi commented Dec 14, 2023 via email

@lobre
Copy link
Contributor Author

lobre commented Dec 14, 2023

That is nice!

I am trying to open neatvi with some files filtered in a pipe with it seems neatvi does not like it though.

find | grep yml | xargs neatvi

@aligrudi
Copy link
Owner

aligrudi commented Dec 16, 2023 via email

@lobre
Copy link
Contributor Author

lobre commented Dec 20, 2023

It needs to read from the tty.

Is there a technical reason why this is the case? What is blocking from having xargs working "normally"?

@aligrudi
Copy link
Owner

aligrudi commented Dec 20, 2023 via email

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

No branches or pull requests

2 participants