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

Change in project state between lint and HTML report may misplace files #31

Closed
TWiStErRob opened this issue Oct 14, 2018 · 1 comment
Closed
Labels
a:bug something isn't working as expected in:report violationReportConsole/violationReportHtml
Milestone

Comments

@TWiStErRob
Copy link
Owner

Lines and files may be moving around, so make generation more lenient and display proper errors for missing stuff.
For example: lint was run with ic_launcher_app and then it was renamed to ic_launcher and then HTML report crashed while reading the outdated lint report.

Caused by: java.io.FileNotFoundException: P:\projects\workspace\ColorFilters\app\src\main\res\drawable-hdpi\ic_launcher_app.png (The system cannot find the file specified)
        at java.io.FileInputStream.open0(Native Method)
        at java.io.FileInputStream.open(FileInputStream.java:195)
        at java.io.FileInputStream.<init>(FileInputStream.java:138)
        at kotlin.io.FilesKt__FileReadWriteKt.readBytes(FileReadWrite.kt:63)
        at net.twisterrob.gradle.quality.report.html.ViolationsGrouperKt$emitViolation$1$3$8.invoke(ViolationsGrouper.kt:128)

Consider also: detect outdated lint reports? (e.g. lint-resuts HTML file is newer than XML)

@TWiStErRob TWiStErRob added the a:feature new feature, improvement or request label Oct 14, 2018
@TWiStErRob TWiStErRob added this to the v0.8 milestone Oct 14, 2018
@TWiStErRob
Copy link
Owner Author

partially fixed at b8375bb#diff-4722b864662b4e530ab1aa7ff44522e3R196

@TWiStErRob TWiStErRob added a:bug something isn't working as expected and removed a:feature new feature, improvement or request labels Feb 16, 2019
@TWiStErRob TWiStErRob added the in:report violationReportConsole/violationReportHtml label Feb 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:bug something isn't working as expected in:report violationReportConsole/violationReportHtml
Projects
None yet
Development

No branches or pull requests

1 participant