Skip to content

SONARJAVA-5849 Migrate test analyze from Cirrus to GitHub actions #5345

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

SONARJAVA-5849 Migrate test analyze from Cirrus to GitHub actions #5345

wants to merge 1 commit into from

Conversation

@tomasz-tylenda-sonarsource
Copy link
Contributor

@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource commented Nov 7, 2025 •
edited by atlassian bot
Loading

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Migrate test analyze from Cirrus to GitHub actions SONARJAVA-5849 Migrate test analyze from Cirrus to GitHub actions Nov 7, 2025
@tomasz-tylenda-sonarsource tomasz-tylenda-sonarsource force-pushed the tt/gha-test-analyze branch 2 times, most recently from e8ed2d6 to 24cda94 Compare November 10, 2025 14:52
@sonarqube-next
Copy link

sonarqube-next bot commented Nov 10, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

GabrielFleischer
GabrielFleischer approved these changes Nov 10, 2025
View reviewed changes
Copy link
Contributor

@GabrielFleischer GabrielFleischer left a comment

Choose a reason for hiding this comment

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

The changes look good to me. Looking at the cirrus code, the migration is faithful.

But i have to admit I don't understand neither the original nor the new process goal.

If you are confident about it, feel free to merge. But adding a bit of documentation might be a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GabrielFleischer GabrielFleischer GabrielFleischer approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tomasz-tylenda-sonarsource @GabrielFleischer