Skip to content

fix: trigger SonarCloud after release and remove redundant main push …#1011

Merged
luluiz merged 1 commit intodevelopfrom
feature/trigger-sonar-after-release
Mar 30, 2026
Merged

fix: trigger SonarCloud after release and remove redundant main push …#1011
luluiz merged 1 commit intodevelopfrom
feature/trigger-sonar-after-release

Conversation

@luluiz
Copy link
Copy Markdown
Member

@luluiz luluiz commented Mar 25, 2026

User-Facing Changes

N/A

Description

After a release, the SonarCloud analysis on main was showing an outdated project version and stale code coverage because:

  1. The release.yml commit uses [skip actions], which skips all workflows — including SonarCloud
  2. The previous push-triggered Sonar analysis ran on the merge commit (before the version bump), so it always reported the old version

This PR fixes the issue by:

  • Adding a step in release.yml to explicitly trigger the SonarCloud workflow on main via gh workflow run after the release is complete — ensuring analysis runs with the correct version
  • Removing main from the push trigger in sonarqube.yml to avoid a redundant (and outdated) Sonar run on every merge to main

Related to #1009

Checklist

  • The web version was tested and it is running ok
  • The desktop version was tested and it is running ok
  • This change is covered by unit tests
  • Files constants.ts, types.ts and *.style.ts have been checked and relevant code snippets have been relocated

@luluiz luluiz self-assigned this Mar 25, 2026
@luluiz luluiz added the cicd For CI/CD changes label Mar 25, 2026
@sonarqubecloud
Copy link
Copy Markdown

@luluiz luluiz merged commit 0deb6c0 into develop Mar 30, 2026
19 checks passed
@luluiz luluiz deleted the feature/trigger-sonar-after-release branch March 30, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cicd For CI/CD changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants