Skip to content

Conversation

@AndyScherzinger
Copy link
Member

Resolves #4767

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@nextcloud-android-bot
Copy link
Collaborator

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11469.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@nextcloud-android-bot
Copy link
Collaborator

Codacy

303

Lint

TypemasterPR
Warnings5959
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings119
Security Warnings44
Dodgy code Warnings138
Total423

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings119
Security Warnings44
Dodgy code Warnings138
Total423

@codecov
Copy link

codecov bot commented Oct 30, 2019

Codecov Report

Merging #4768 into master will decrease coverage by 0.07%.
The diff coverage is 20%.

@@             Coverage Diff              @@
##             master    #4768      +/-   ##
============================================
- Coverage     17.55%   17.48%   -0.08%     
  Complexity        3        3              
============================================
  Files           376      376              
  Lines         32383    32383              
  Branches       4568     4568              
============================================
- Hits           5685     5662      -23     
- Misses        25780    25801      +21     
- Partials        918      920       +2
Impacted Files Coverage Δ Complexity Δ
...ncloud/android/ui/adapter/ActivityListAdapter.java 3.63% <0%> (ø) 0 <0> (ø) ⬇️
...in/java/com/owncloud/android/utils/ThemeUtils.java 51.57% <0%> (ø) 0 <0> (ø) ⬇️
...ndroid/ui/activity/SsoGrantPermissionActivity.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...ud/android/ui/adapter/NotificationListAdapter.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...m/owncloud/android/ui/activity/DrawerActivity.java 45.19% <100%> (ø) 0 <0> (ø) ⬇️
.../owncloud/android/ui/activity/ToolbarActivity.java 60.86% <0%> (-2.9%) 0% <0%> (ø)
...loud/android/datamodel/ThumbnailsCacheManager.java 33.39% <0%> (-2.74%) 0% <0%> (ø)
.../third_parties/daveKoeller/AlphanumComparator.java 80.95% <0%> (-1.2%) 0% <0%> (ø)
...cloud/android/ui/activity/FileDisplayActivity.java 20.01% <0%> (-0.25%) 0% <0%> (ø)
...owncloud/android/ui/adapter/OCFileListAdapter.java 29.77% <0%> (-0.2%) 0% <0%> (ø)
... and 2 more

@AndyScherzinger AndyScherzinger merged commit 946e5c2 into master Oct 31, 2019
@delete-merged-branch delete-merged-branch bot deleted the colorNames branch October 31, 2019 07:52
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.10.0 milestone Oct 31, 2019
tobiasKaminsky added a commit that referenced this pull request Nov 1, 2019
e3280b0 Merge pull request #4775 from nextcloud/dependabot/gradle/com.h3xstream.findsecbugs-findsecbugs-plugin-1.10.1
575cd01 Bump findsecbugs-plugin from 1.10.0 to 1.10.1
946e5c2 Merge pull request #4768 from nextcloud/colorNames
78ed97c [tx-robot] updated from transifex
f77b5bd daily dev 20191031
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.

Improve color name pattern

4 participants