Skip to content

fix: disable Sonarcloud for external PRs#420

Merged
cameri merged 1 commit intocameri:mainfrom
Justxd22:fix/disable-pr-sonar
Apr 9, 2026
Merged

fix: disable Sonarcloud for external PRs#420
cameri merged 1 commit intocameri:mainfrom
Justxd22:fix/disable-pr-sonar

Conversation

@Justxd22
Copy link
Copy Markdown
Contributor

@Justxd22 Justxd22 commented Apr 9, 2026

Updated Sonarcloud so it does not run for fork PRs

Effect:

  • push and internal PRs: SonarCloud still runs.
  • PRs from external forks: SonarCloud job is skipped, so it won’t fail due to missing secrets.

Related Issue

#402

Types of changes

  • Non-functional change (docs, style, minor refactor)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my code changes.
  • All new and existing tests passed.

@Justxd22 Justxd22 mentioned this pull request Apr 9, 2026
@cameri cameri merged commit 3879b07 into cameri:main Apr 9, 2026
10 checks passed
@cameri
Copy link
Copy Markdown
Owner

cameri commented Apr 9, 2026

Thank you!!!

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.

2 participants