So I got this from sonarcloud. It identifies an unused variable for sure.
Doesn't help me understand who or why it got into testing.
Question: shouldn't our scala compiler settings be giving us a warning about
this, and aren't warnings fatal per our current practices?
I wouldn't expect we'd need solarcloud to find this for us. I'd expect every
developer to see these on every compile.
________________________________
From: SonarCloud <nore...@sonarcloud.io>
Sent: Monday, October 26, 2020 2:45 PM
To: Beckerle, Mike <mbecke...@owlcyberdefense.com>
Subject: [SonarCloud] incubator-daffodil: 1 new issue (new debt: 5min)
Project: incubator-daffodil
Version: not provided
1 new issue (new debt: 5min)
Type
Bug: 0 Vulnerability: 0 Code Smell: 1
Rules
Unused local variables should be removed (scala): 1
Most impacted files
ConvertTextNumberUnparser.scala: 1
More details at:
https://sonarcloud.io/project/issues?id=apache_incubator-daffodil&createdAt=2020-10-26T18%3A44%3A35%2B0000
