I would try to handle them in several small PRs either grouped by module or by warning type.
On Fri, May 27, 2022 at 9:24 AM Eric Pugh <ep...@opensourceconnections.com> wrote: > Hey all, was poking around at a unit test while watching TV and noticed > lots of warnings from IntelliJ, little stuff like exceptions being thrown > that don’t need to be thrown, unused variables, or typos. > > I was thinking about going through and fixing those, just to get the long > list of problems down…. Is this best handled with a single PR with a JIRA > issue? Say a JIRA issue like “Clean up IntelliJ warnings for Test Code”? > > > Eric > > > > _______________________ > *Eric Pugh **| *Founder & CEO | OpenSource Connections, LLC | 434.466.1467 > | http://www.opensourceconnections.com | My Free/Busy > <http://tinyurl.com/eric-cal> > Co-Author: Apache Solr Enterprise Search Server, 3rd Ed > <https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw> > This e-mail and all contents, including attachments, is considered to be > Company Confidential unless explicitly stated otherwise, regardless > of whether attachments are marked as such. > >
