Skip to content

chore: exclude .ci/ from SonarCloud source analysis#5043

Merged
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:mainfrom
zdrapela:chore/exclude-ci-from-sonarcloud
Jul 2, 2026
Merged

chore: exclude .ci/ from SonarCloud source analysis#5043
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:mainfrom
zdrapela:chore/exclude-ci-from-sonarcloud

Conversation

@zdrapela

@zdrapela zdrapela commented Jul 2, 2026

Copy link
Copy Markdown
Member

The .ci/ directory contains CI infrastructure (shell scripts, Dockerfiles, YAML configs) that is not application source code. SonarCloud findings on these files are noise.

Adds .ci/**/* to sonar.exclusions in .sonarcloud.properties.

Suggested in #5036 (comment).

The .ci/ directory contains CI infrastructure (shell scripts,
Dockerfiles, YAML configs) that is not application source code.
SonarCloud findings on these files are noise.

Assisted-by: OpenCode
@openshift-ci openshift-ci Bot requested review from durandom and josephca July 2, 2026 07:57
@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

@rm3l rm3l left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci openshift-ci Bot added the lgtm label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.77%. Comparing base (830ac72) to head (f23dbee).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5043      +/-   ##
==========================================
- Coverage   55.39%   54.77%   -0.62%     
==========================================
  Files         122      110      -12     
  Lines        2365     2147     -218     
  Branches      564      537      -27     
==========================================
- Hits         1310     1176     -134     
+ Misses       1048      970      -78     
+ Partials        7        1       -6     
Flag Coverage Δ
rhdh 54.77% <ø> (-0.62%) ⬇️

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 830ac72...f23dbee. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rm3l

rm3l commented Jul 2, 2026

Copy link
Copy Markdown
Member

/override e2e-ocp-helm

Unrelated

@openshift-ci

openshift-ci Bot commented Jul 2, 2026

Copy link
Copy Markdown

@rm3l: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • e2e-ocp-helm

Only the following failed contexts/checkruns were expected:

  • Auto-Approve and Label PRs
  • Build Image
  • Build with Node.js
  • Test with Node.js
  • ci/prow/e2e-ocp-helm
  • pull-ci-redhat-developer-rhdh-main-e2e-ocp-helm
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

Details

In response to this:

/override e2e-ocp-helm

Unrelated

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@rm3l

rm3l commented Jul 2, 2026

Copy link
Copy Markdown
Member

/override ci/prow/e2e-ocp-helm

@openshift-ci

openshift-ci Bot commented Jul 2, 2026

Copy link
Copy Markdown

@rm3l: Overrode contexts on behalf of rm3l: ci/prow/e2e-ocp-helm

Details

In response to this:

/override ci/prow/e2e-ocp-helm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-merge-bot openshift-merge-bot Bot merged commit 5c7ffe3 into redhat-developer:main Jul 2, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants