Skip to content

build: exclude binary assets from SonarCloud analysis - #763

Merged
setchy merged 4 commits into
mainfrom
build/sonarcloud-ci
Oct 3, 2026
Merged

setchy merged 4 commits into
mainfrom
build/sonarcloud-ci

Conversation

@setchy

@setchy setchy commented Oct 3, 2026

Copy link
Copy Markdown
Member

Summary

Exclude binary asset files from SonarCloud's source scope.

 sonar.sources=./src
+sonar.exclusions=**/*.png, **/*.jpg, **/*.jpeg, **/*.gif, **/*.webp, **/*.avif, **/*.ico, **/*.bmp

Why

src/assets/*.png sit inside sonar.sources and get indexed as source files, causing scanner warnings like:

Invalid character encountered in file .../src/assets/notifications.png at line 1 for encoding UTF-8.

Binary assets can't be analyzed, so they're excluded instead.

@setchy
setchy requested a review from afonsojramos as a code owner October 3, 2026 20:36
@netlify

netlify Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for gitify ready!

Name Link
🔨 Latest commit 4d20cf0
🔍 Latest deploy log https://app.netlify.com/projects/gitify/deploys/6ac169991e41fe00086fa8b7
😎 Deploy Preview https://deploy-preview-763--gitify.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 88 (🔴 down 1 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

# Conflicts:
#	README.md
#	sonar-project.properties
@setchy
setchy merged commit c54d16e into main Oct 3, 2026
12 checks passed
@setchy
setchy deleted the build/sonarcloud-ci branch October 3, 2026 20:46
@sonarqubecloud

sonarqubecloud Bot commented Oct 3, 2026

Copy link
Copy Markdown

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.

1 participant