gerlowskija commented on code in PR #975: URL: https://github.com/apache/solr/pull/975#discussion_r965335669
########## solr/core/src/java/org/apache/solr/core/CoreContainer.java: ########## @@ -16,12 +16,51 @@ */ package org.apache.solr.core; +import static java.util.Objects.requireNonNull; Review Comment: This is driving me nuts. I'm not sure if it's just my editor settings but it seems like half the LOC in any given PR is just import reordering or changes. I would've thought spotless would standardize this but apparently not? No reason we shouldn't have a standard here if we're already making people run spotless. Not that I love spotless either, given how it's mangled every comment in this PR 😉 But at least it's something. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
