Skip to content

Small improvement in update library script - #17884

Open
daniele-verducci wants to merge 2 commits into
masterfrom
improv/updateLibraryHash
Open

daniele-verducci wants to merge 2 commits into
masterfrom
improv/updateLibraryHash

Conversation

@daniele-verducci

Copy link
Copy Markdown
Contributor

Two very small improvements to updateLibraryHash script:

  • Early fail with clear message on updateLibraryHash to prevent confusion
  • Prevent script from failing it running on a macine with the git hooks configured as documented in CONTRIBUTING.md

Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
… configured as documented in CONTRIBUTING.md

Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
@daniele-verducci
daniele-verducci force-pushed the improv/updateLibraryHash branch from 9278aed to 7e321ad Compare October 26, 2026 10:02
@codecov

codecov Bot commented Oct 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions

Copy link
Copy Markdown

📱 QA build

Download app-qa-debug.apk
QR code Open the QR code for this download
Commit 7e321ad
Version 17884
Available until 7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice2424
Correctness5151
Dodgy code158158
Experimental11
Malicious code vulnerability11
Multithreaded correctness2020
Performance2828
Security1515
Total298298

This branch has not been deployed

No deployments
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.

1 participant