Skip to content

fix(ci): reuse Clang build for clang-tidy - #10696

Merged
Rello merged 1 commit into
masterfrom
codex/ci-reuse-clang-build
Sep 1, 2026
Merged

fix(ci): reuse Clang build for clang-tidy#10696
Rello merged 1 commit into
masterfrom
codex/ci-reuse-clang-build

Conversation

@Rello

@Rello Rello commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Export compile commands from the existing Linux Clang build, then run the changed-line clang-tidy analysis after its test suite. Remove the standalone workflow that rebuilt the same source tree.

Estimated saving: about 12 minutes and one job per update.

Assisted-by: Codex:GPT-5

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

馃挕 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d749939a75

鈩癸笍 About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 馃憤.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/linux-clang-compile-tests.yml
Comment thread .github/workflows/linux-clang-compile-tests.yml
@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:98c1f94c090677c5a76b4264448ee4f23deab6b6485558eb7edd44e221bfba8f

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.

Export compile commands from the existing Linux Clang build, then run the changed-line clang-tidy analysis after its test suite. Remove the standalone workflow that rebuilt the same source tree.

Signed-off-by: Rello <github@scherello.de>
Assisted-by: Codex:GPT-5
@mgallien
mgallien enabled auto-merge August 31, 2026 15:16
@mgallien
mgallien force-pushed the codex/ci-reuse-clang-build branch from d749939 to eb62a20 Compare August 31, 2026 15:16
@sonarqubecloud

Copy link
Copy Markdown

@Rello

Rello commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0

@Rello
Rello disabled auto-merge September 1, 2026 07:00
@Rello
Rello merged commit 303fc3b into master Sep 1, 2026
19 of 21 checks passed
@Rello
Rello deleted the codex/ci-reuse-clang-build branch September 1, 2026 07:00
@backportbot

backportbot Bot commented Sep 1, 2026

Copy link
Copy Markdown

The backport to stable-34.0 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable-34.0
git pull origin stable-34.0

# Create the new backport branch
git checkout -b backport/10696/stable-34.0

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick eb62a20a

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/10696/stable-34.0

Error: Failed to check for changes with origin/stable-34.0: No changes found in backport branch


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

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