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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document vec_sep2 for cli_vec(style) #677

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ddsjoberg
Copy link

Hello @gaborcsardi ! I love this package! 馃ぉ

I was modifying .val function to list elements with an "or" at the end instead of "and", e.g. cli::cli_text("{.val {letters[1:3]}}"). Everything was going smoothly until I passed a vector of size two, and my "or" reverted back to "and". I was pretty confused, but after searching the internals, I found the style element vec_sep2 which quickly solved my issue.

This PR simply adds vec_sep2 to the cli_vec() help file to make it a bit easier to find for users. I think others may benefit from this as well?

(Apologies, for not re-documenting the roxygen comments....I was running into an error.)

Thank you!

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

Successfully merging this pull request may close these issues.

None yet

1 participant