Skip to content

Conversation

@tobiasKaminsky
Copy link
Member

As suggested in #994

Thanks @bglass 🎉

Signed-off-by: tobiasKaminsky [email protected]

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

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 Nov 16, 2022

Codecov Report

Merging #997 (b8cdd21) into master (d2b81dd) will increase coverage by 0.14%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #997      +/-   ##
==========================================
+ Coverage   48.95%   49.10%   +0.14%     
==========================================
  Files         178      178              
  Lines        6678     6678              
  Branches      884      884              
==========================================
+ Hits         3269     3279      +10     
+ Misses       2939     2931       -8     
+ Partials      470      468       -2     
Impacted Files Coverage Δ
...ncloud/android/lib/common/network/WebdavEntry.java 79.28% <0.00%> (+0.39%) ⬆️
...om/owncloud/android/lib/common/OwnCloudClient.java 44.00% <0.00%> (+1.14%) ⬆️
...d/lib/common/operations/RemoteOperationResult.java 46.97% <0.00%> (+1.15%) ⬆️
...lib/resources/files/RemoveFileRemoteOperation.java 88.88% <0.00%> (+16.66%) ⬆️

@AlvaroBrey AlvaroBrey merged commit b428f94 into master Nov 16, 2022
@delete-merged-branch delete-merged-branch bot deleted the fixSampleClient branch November 16, 2022 20:01
@AlvaroBrey AlvaroBrey added this to the NC Android Lib 2.13.0 milestone Nov 16, 2022
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.

3 participants