Skip to content

Conversation

@mdelapenya
Copy link
Member

@mdelapenya mdelapenya commented Mar 5, 2025

  • choe(ci): use new sonarqube GH action
  • revert: for testing on CI

What does this PR do?

It uses the new SonarQube GH action, as the old one was deprecated. It generates the sonar properties on the fly, removing the need of maintaining a properties file with all the modules. As a result, the code generation tool for the sonar properties is removed in this PR.

Also, the project in Sonar will be a monorepo, having as project key:

  • core: testcontainers_testcontainers-go
  • modules: testcontainers_testcontainers-go_modules_postgres

and as project name:

  • core: testcontainers-go
  • modules: testcontainers-go-modules-postgres

As a consequence, the sonar step will be executed per module, updating each project of the parent project with the given changes.

Why is it important?

Sonar is failing since March 4th 2025. Besides that, simplifying the maintenance of modules and collecting sonar metrics per module, not per monorepo.

@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 2d3d0de
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/67d3128c63ad740008bfa6a8
😎 Deploy Preview https://deploy-preview-3007--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

* main:
  chore(deps): bump github.com/opencontainers/image-spec from 1.1.0 to 1.1.1, dario.cat/mergo from 1.0.0 to 1.0.1 (testcontainers#3030)
  chore(deps): bump github/codeql-action from 3.28.0 to 3.28.11 (testcontainers#3014)
  chore(deps): bump ossf/scorecard-action from 2.4.0 to 2.4.1 (testcontainers#3013)
  chore: readd dependabot, including a way to refresh the project files for all the modules (testcontainers#2997)
  docs: refine texts on how to set the module image (testcontainers#3012)
  feat(modules): add dind module (testcontainers#3004)
  docs: correct container variable (testcontainers#3010)
@mdelapenya mdelapenya self-assigned this Mar 13, 2025
@mdelapenya mdelapenya requested a review from stevenh March 13, 2025 17:09
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Mar 13, 2025
@mdelapenya mdelapenya marked this pull request as ready for review March 13, 2025 17:09
@mdelapenya mdelapenya requested a review from a team as a code owner March 13, 2025 17:09
@mdelapenya
Copy link
Member Author

I'm going to merge this one and explore how it goes in the Sonarcloud instance.

@mdelapenya mdelapenya merged commit c1615e7 into testcontainers:main Mar 14, 2025
184 checks passed
@mdelapenya mdelapenya deleted the sonar-fix branch March 14, 2025 16:16
mdelapenya added a commit that referenced this pull request Mar 17, 2025
…er-28.0.1incompatible

* main:
  chore!: export DockerCompose type in compose package (#2953)
  chore(deps): bump golang.org/x/net from 0.33.0 to 0.36.0 in /modules (#3033)
  chore(sonar): set main as default branch (#3038)
  fix(sonar): run on ubuntu and the lowest Go version (#3037)
  fix(ci): update core path in sonar build (#3036)
  fix: use empty dir for the core (#3035)
  fix: pass secret to the reusable workflow (#3034)
  chore(ci): use new GH action for SonarQube (#3007)
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Mar 17, 2025
* main:
  chore(deps)!: bump github.com/docker/docker from 27.1.1+incompatible to 28.0.1+incompatible (testcontainers#3017)
  chore!: export DockerCompose type in compose package (testcontainers#2953)
  chore(deps): bump golang.org/x/net from 0.33.0 to 0.36.0 in /modules (testcontainers#3033)
  chore(sonar): set main as default branch (testcontainers#3038)
  fix(sonar): run on ubuntu and the lowest Go version (testcontainers#3037)
  fix(ci): update core path in sonar build (testcontainers#3036)
  fix: use empty dir for the core (testcontainers#3035)
  fix: pass secret to the reusable workflow (testcontainers#3034)
  chore(ci): use new GH action for SonarQube (testcontainers#3007)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Changes that do not impact the existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant