-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add a few cleanup formatters #853
Conversation
Readd functionality "Remove double braces around BibTeX fields when loading" (which was removed in c2928d2) as a formatter.
3803323
to
aa91be1
Compare
@@ -25,11 +25,6 @@ public void testBasic() { | |||
} | |||
|
|||
@Test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep and add (expected...)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, now I saw the next file... ;-)
👍 Now I only want a formatter that removes ' around fields. A colleague does that as that also keeps the casing... However, I guess it may be a rare use case for the general public... |
👍 nice. I especially like the data driven test of all formatters! Good idea. |
LGTM 👍 |
Add a few cleanup formatters
This PR adds the following formatters, which are suitable for the new save actions functionality #821:
loading" but was removed from the parser in c2928d2.
Also add some tests for all formatters.