Skip to content
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

Fix #182 LizardSensor logs full stack trace exception instead warning if report file not exists #183

Merged
merged 1 commit into from
Aug 24, 2019

Conversation

nfalco79
Copy link

Add a check that if the report file does not exists than show a warning.

Some details:

  • Instead contact string paths using File.separator I had use the constructor of File(parent, child) that do the work
  • I check is reportFile is file that also check is exists.

… if report file not exists

Add a check that if the report file does not exists than show a warning.
@nfalco79
Copy link
Author

@zippy1978 any chance to merge this fix?

@Quesar
Copy link

Quesar commented Feb 27, 2019

A merge of this fix would be nice, because the annoying stack trace for java.io.FileNotFoundException pops up even when running analysis on a Java project.

@gaelfoppolo gaelfoppolo removed the request for review from davidy4ng August 24, 2019 13:30
@gaelfoppolo gaelfoppolo merged commit 62ab06c into Idean:develop Aug 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants