PAINTROID-431 deleting unused file log.java

[akebu6]

Refactored the log test in test/java/android/util from Java to Kotlin. Then, later realised file was never in use from the original nor the new branch and hence deleted the file

Ticket: https://jira.catrob.at/browse/PAINTROID-431

Your checklist for this pull request

Please review the contributing guidelines and wiki pages of this repository.

• Include the name of the Jira ticket in the PR’s title
• Include a summary of the changes plus the relevant context
• Choose the proper base branch (develop)
• Confirm that the changes follow the project’s coding guidelines
• Verify that the changes generate no compiler or linter warnings
• Perform a self-review of the changes
• Verify to commit no other files than the intentionally changed ones
• Include reasonable and readable tests verifying the added or changed behavior
• Confirm that new and existing unit tests pass locally
• Check that the commits’ message style matches the project’s guideline
• Stick to the project’s gitflow workflow
• Verify that your changes do not have any conflicts with the base branch
• After the PR, verify that all CI checks have passed
• Post a message in the #paintroid Slack channel and ask for a code reviewer

[FromG]

fix static warnings on jenkins please.

[akebu6]

Got it, will do

[akebu6]

Any idea how I can fix the static warnings? I noticed them but wasn't sure how to go about fixing them

[FromG]

first you click on the details

then you have this option where it shows how many static warnings you have

when you click on the static warnings you will get to the different errors which occured.
by clicking on the errors you can then see which files are affected.

if you further click on the file. you can then go to the exact line in the code.

here you can exactly see the warning in the code.

in this example ExpressionBodySyntax you should write it like this:
fun d(tag: String, msg: String): Int = 0
i think there is also a option for converting to expressionbodysyntax in android studio.

Furthermore you will also be able to see the tests which failed on jenkins.
example from a different pr:

these will only appear after your static warnings are resolved.

[akebu6]

all right, let me work on them

[akebu6]

@FromG the changes requested have been worked on, tests pass locally and on Jerkins. The file was deleted as it had no usages in both the develop branch and the new branch

[ThorstenBandel]

Thanks!

Changes done