Skip to content

Conversation

@mgallien
Copy link
Collaborator

@mgallien mgallien commented Dec 4, 2025

should make it easier to discover when needed

@mgallien mgallien added this to the 4.1.0 milestone Dec 4, 2025
@mgallien mgallien requested a review from nilsding as a code owner December 4, 2025 14:54
Copy link
Member

@nilsding nilsding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

I wonder if we should try to include that crash log in debug archives

should make it easier to discover when needed

Signed-off-by: Matthieu Gallien <[email protected]>
@mgallien mgallien force-pushed the bugfix/betterLogForCrashLog branch from 7cf1c65 to 485fff0 Compare December 5, 2025 08:11
@mgallien mgallien enabled auto-merge December 5, 2025 08:11
@github-actions
Copy link

github-actions bot commented Dec 5, 2025

Artifact containing the AppImage: nextcloud-appimage-pr-9187.zip

Digest: sha256:ccc0e9a5fbcc41d596006d88e7e1fa5746223614342d5120da15051460fdd328

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 5, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
10 New Code Smells (required ≤ 0)
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@mgallien mgallien merged commit d1c09cb into master Dec 5, 2025
21 of 22 checks passed
@mgallien mgallien deleted the bugfix/betterLogForCrashLog branch December 5, 2025 08:50
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