Skip to content

Conversation

@tobiasKaminsky
Copy link
Member

@tobiasKaminsky tobiasKaminsky commented Jan 5, 2023

Detekt is now at 0 💪

Signed-off-by: tobiasKaminsky [email protected]

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2023

SpotBugs (new)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

SpotBugs (master)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

@codecov
Copy link

codecov bot commented Jan 5, 2023

Codecov Report

Merging #1027 (16027d9) into master (93812ac) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

❗ Current head 16027d9 differs from pull request most recent head ac81c09. Consider uploading reports for the commit ac81c09 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1027      +/-   ##
==========================================
- Coverage   49.42%   49.27%   -0.15%     
==========================================
  Files         179      179              
  Lines        6727     6725       -2     
  Branches      894      892       -2     
==========================================
- Hits         3325     3314      -11     
- Misses       2935     2942       +7     
- Partials      467      469       +2     
Impacted Files Coverage Δ
.../main/java/com/nextcloud/common/NextcloudClient.kt 58.82% <0.00%> (+1.68%) ⬆️
...wncloud/android/lib/resources/files/FileUtils.java 71.42% <0.00%> (-14.29%) ⬇️
...b/resources/files/CreateFolderRemoteOperation.java 70.58% <0.00%> (-8.83%) ⬇️
...d/lib/common/operations/RemoteOperationResult.java 45.82% <0.00%> (-1.16%) ⬇️
...om/owncloud/android/lib/common/OwnCloudClient.java 42.85% <0.00%> (-1.15%) ⬇️

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2023

SpotBugs (new)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

SpotBugs (master)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

@AlvaroBrey AlvaroBrey self-requested a review January 7, 2023 13:17
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2023

SpotBugs (new)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

SpotBugs (master)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

tobiasKaminsky and others added 2 commits January 9, 2023 13:14
do no allow any issues on detekt

Signed-off-by: tobiasKaminsky <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2023

SpotBugs (new)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

SpotBugs (master)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2023

SpotBugs (new)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

SpotBugs (master)

Warning Type Number
Bad practice Warnings 10
Correctness Warnings 35
Internationalization Warnings 6
Malicious code vulnerability Warnings 43
Multithreaded correctness Warnings 3
Performance Warnings 12
Security Warnings 1
Dodgy code Warnings 31
Total 141

@nextcloud-android-bot
Copy link
Collaborator

@AlvaroBrey AlvaroBrey disabled auto-merge January 9, 2023 13:41
@AlvaroBrey AlvaroBrey merged commit 0af404a into master Jan 9, 2023
@AlvaroBrey AlvaroBrey added this to the NC Android Lib 2.14.0 milestone Jan 9, 2023
@delete-merged-branch delete-merged-branch bot deleted the detekt branch January 9, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants