Skip to content

fix(ci): skip unrelated component workflows - #10695

Merged
Rello merged 2 commits into
masterfrom
codex/ci-path-filters
Aug 31, 2026
Merged

fix(ci): skip unrelated component workflows#10695
Rello merged 2 commits into
masterfrom
codex/ci-path-filters

Conversation

@Rello

@Rello Rello commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Restrict the independent macOS workflow triggers to their own paths. Avoid Sonar analysis for translation-only, documentation-tree, and Markdown-only pushes to master.

  • Unrelated PR updates avoid three jobs: mac-crafter, FileProviderKit lint, and FileProviderKit tests.
  • Observed saving: approximately 7m31s of runner time per unrelated PR update.
    • The latest translation-only master push would have avoided all four triggered jobs, saving 29m45s total.
    • SonarCloud alone accounted for 22m25s of that push.

Assisted-by: Codex:GPT-5

Restrict the independent macOS workflow triggers to their own paths. Avoid Sonar analysis for translation-only, documentation-tree, and Markdown-only pushes to master.

Signed-off-by: Rello <github@scherello.de>
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: fadc96ed5b

鈩癸笍 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/mac-crafter-ci.yml
Comment thread .github/workflows/sonarcloud.yml Outdated
@Rello

Rello commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator Author

...there is a clang commit. this is wrong here. I will remove it...

@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: 5946fcee4f

鈩癸笍 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/sonarcloud.yml
Run SonarCloud on master pushes only when CMake, C, or C++ source files change.

Co-authored-by: Jyrki Gadinger <nilsding@nilsding.org>
Signed-off-by: Rello <github@scherello.de>
Assisted-by: Codex:GPT-5
@Rello
Rello force-pushed the codex/ci-path-filters branch from 5946fce to 4356375 Compare August 31, 2026 11:38
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

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

Digest: sha256:5ed6307a0c8c9fef5401968ea1e6ac44cee38249b66c67f50bfa26beb2686a82

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.

@Rello
Rello merged commit c629357 into master Aug 31, 2026
25 of 26 checks passed
@Rello
Rello deleted the codex/ci-path-filters branch August 31, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants