Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

adding keep new lines as parameter #179

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tzahush
Copy link

@tzahush tzahush commented Feb 8, 2024

I would like to add the option to keep new lines as a flag

@tzahush tzahush changed the title adding keep new lines adding keep new lines as parameter Feb 8, 2024
@tzahush
Copy link
Author

tzahush commented Feb 12, 2024

@chrusty Do you think it is possible to get a review for this?
Super simple PR which will be great to get merged

@chrusty
Copy link
Owner

chrusty commented Feb 12, 2024

@chrusty Do you think it is possible to get a review for this? Super simple PR which will be great to get merged

Hi @tzahush. I'm not sure this PR actually does anything in its current state - it appears to just set a flag!

@tzahush
Copy link
Author

tzahush commented Feb 12, 2024

@chrusty today all flags can be set using CLI except for this flag, since we are using the CLI it'll be very helpful to be able to align it so all flags can be set with CLI.

@tzahush
Copy link
Author

tzahush commented Feb 16, 2024

@chrusty any update?
The main purpose is for people (specifically my team in this case :) ) to be able to use this flag that is already set in the system using the CLI.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants