Skip to content

test for #5039#7504

Closed
tobiasKaminsky wants to merge 1 commit into
stable-3.14from
testFor5039
Closed

test for #5039#7504
tobiasKaminsky wants to merge 1 commit into
stable-3.14from
testFor5039

Conversation

@tobiasKaminsky

@tobiasKaminsky tobiasKaminsky commented Nov 26, 2020

Copy link
Copy Markdown
Member

#5039

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

Testing

Writing tests is very important. Please try to write some tests for your PR.
If you need help, please do not hesitate to ask in this PR for help.

unit tests
instrumented tests
UI tests

  • Tests written, or not not needed

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Lint

Typestable-3.14PR
Warnings307307
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings27
Correctness Warnings52
Internationalization Warnings9
Multithreaded correctness Warnings9
Performance Warnings73
Security Warnings41
Dodgy code Warnings100
Total311

SpotBugs (stable-3.14)

Warning TypeNumber
Bad practice Warnings27
Correctness Warnings52
Internationalization Warnings9
Multithreaded correctness Warnings9
Performance Warnings73
Security Warnings41
Dodgy code Warnings100
Total311

@AndoKarim

Copy link
Copy Markdown

APK not available.
Tried to redo a PR to trigger android-bot #7625 but it didn't.

@AndyScherzinger
AndyScherzinger marked this pull request as ready for review December 7, 2020 20:15
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/58.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.

@AndyScherzinger

Copy link
Copy Markdown
Member

@AndoKarim I just did a rebase, so here is the latest APK based on this PR: #7504 (comment)

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Lint

Typestable-3.14PR
Warnings307307
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings27
Correctness Warnings52
Internationalization Warnings9
Multithreaded correctness Warnings9
Performance Warnings73
Security Warnings41
Dodgy code Warnings100
Total311

SpotBugs (stable-3.14)

Warning TypeNumber
Bad practice Warnings27
Correctness Warnings52
Internationalization Warnings9
Multithreaded correctness Warnings9
Performance Warnings73
Security Warnings41
Dodgy code Warnings100
Total311

@AndyScherzinger

Copy link
Copy Markdown
Member

@tobiasKaminsky I think this PR is obsolete and has been replaced by: #7694 and #7691

@AndyScherzinger AndyScherzinger added the needs info Waiting for info from user(s). Issues with this label will auto-stale. label Jan 5, 2021
@AndyScherzinger

Copy link
Copy Markdown
Member

Fixed on master already and shipped so closing since 3.14. is obsolete

@AndyScherzinger
AndyScherzinger deleted the testFor5039 branch February 3, 2021 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing needs info Waiting for info from user(s). Issues with this label will auto-stale.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants