Skip to content
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.

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

Get SonarCube tracking net code from latest release (unstable branch) #2147

Closed
wants to merge 7 commits into from

Conversation

aleksraiden
Copy link
Contributor

@aleksraiden aleksraiden commented Mar 9, 2024

Add a latest release branch as sonar.newCode.referenceBranch to track a new code, added after latest release. Closes #2058, #2059

Add a latest release branch as sonar.newCode.referenceBranch to track a new code. Closes #2058
@aleksraiden aleksraiden changed the title Get SonarCube tracking net code from latest release(2.8.0) Get SonarCube tracking net code from latest release (unstable branch) Mar 10, 2024
@jihuayu
Copy link
Member

jihuayu commented Mar 11, 2024

I am testing this modification on my test project to see if it takes effect. This will take some time, please wait.

@jihuayu
Copy link
Member

jihuayu commented Mar 11, 2024

I have a try, but it doesn't work in my repo. Does it work in your repo? @aleksraiden

Following the doc, It seems like the sonar.newCode.referenceBranch option is only supported by SonarQube. In SonarCloud we don't have this option.

SonarCloud only version and Number of days.
Maybe we can use the Specific date option, and use the last 30 days' code as new code in sonarcloud setting. @git-hulk
This option can fixed #2059.

And the #2058 is another problem. I am trying it in my repo.

Refer:

@aleksraiden aleksraiden marked this pull request as draft March 11, 2024 17:54
@aleksraiden
Copy link
Contributor Author

@jihuayu thanks about your investigation, you are absolutely correct, I see a docs about sonar cube, not a cloud version.

@jihuayu
Copy link
Member

jihuayu commented Mar 16, 2024

@aleksraiden
Thank you for your help.
I am looking into this issue. Just a few more questions to deal.

Copy link

sonarcloud bot commented Mar 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@PragmaTwice
Copy link
Member

PragmaTwice commented Nov 7, 2024

We are cleaning branches of this repo and this branch seems stale for a long time.

I'll close it and delete this branch. Feel free to submit a new patch if you want.

Also, if you want to keep these branches for a long time, you can push them into your own fork.

@PragmaTwice PragmaTwice closed this Nov 7, 2024
@PragmaTwice PragmaTwice deleted the aleksraiden-sonarcube-new-code branch November 7, 2024 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci area ci
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SonarCloud analysis run on PR branch has wrong coverage on New Code
4 participants