Github user nickwallen commented on a diff in the pull request:
@@ -40,8 +45,8 @@
Context context = new Context.Builder()
- .with(Context.Capabilities.SHELL_VARIABLES , () -> variables)
+ .with(Context.Capabilities.SHELL_VARIABLES , () ->
--- End diff --
Right now, I never let my IDE reformat for me. Like you said, if we get
the code base matching check style and I can load that style into my IDE, then
I'd gladly let it do most of the work for me.
Maybe I'll open a discuss thread. I don't know how to handle this kind of
thing and it happens all the time.
But for this specific scenario in your PR, it really doesn't matter either
way. I think you're good to go either way.