Skip to content

feat: Change newline_separated support multiple newlines#84

Merged
JeffFaer merged 2 commits intogoogle:mainfrom
joesun-google:newlines
Aug 14, 2025
Merged

feat: Change newline_separated support multiple newlines#84
JeffFaer merged 2 commits intogoogle:mainfrom
joesun-google:newlines

Conversation

@joesun-google
Copy link
Contributor

  1. Change the option newline_separated to a new IntOrBool type that treat no=0 and yes=1,
  2. Change the implementation to check for and add N newlines, when newline_separated=N is specified,
  3. Add new test cases.

Tested :

$ go test ./...
?       github.com/google/keep-sorted   [no test files]
?       github.com/google/keep-sorted/cmd       [no test files]
ok      github.com/google/keep-sorted/goldens   (cached)
ok      github.com/google/keep-sorted/keepsorted        (cached)

@JeffFaer JeffFaer enabled auto-merge (squash) August 14, 2025 16:40
@JeffFaer JeffFaer merged commit debebf9 into google:main Aug 14, 2025
5 checks passed
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.

2 participants