Skip to content

chore(PLT-3524): remove SonarCloud setup#746

Open
tf-seti wants to merge 2 commits into
mainfrom
chore/remove-sonarcloud
Open

chore(PLT-3524): remove SonarCloud setup#746
tf-seti wants to merge 2 commits into
mainfrom
chore/remove-sonarcloud

Conversation

@tf-seti

@tf-seti tf-seti commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Jira link: https://typeform.atlassian.net/browse/PLT-3524

SonarCloud is being decommissioned as our unit-test coverage tool.
See the "Measure Unit Test Coverage" Notion page for context.

This PR removes:

  • sonar-project.properties (and .sonarcloud.properties if present)
  • The sonar-cloud-scan job and its coverage-artifact upload step in .github/workflows/*
  • Any SonarCloud/SonarQube badges in README.md

Follow-ups (handled separately by admins):

  • Removing the SONAR_CLOUD_TOKEN repo secret
  • Archiving the project in SonarCloud

Generated with Sourcegraph Batch Changes.

Created by Sourcegraph batch change jhuliano.moreno/remove-sonarcloud.

@tf-seti tf-seti requested a review from a team as a code owner July 15, 2026 10:59
@pr-auditor

pr-auditor Bot commented Jul 15, 2026

Copy link
Copy Markdown

✅ Security Analysis Results

Great news! No security issues found in this pull request.

Analysis Summary:

  • 📁 Files reviewed: 2
  • ✅ No security vulnerabilities detected

💡 Trigger a new security scan by commenting @pr-auditor rescan on this PR.

Security analysis powered by Claude Sonnet 4.6 via pr-auditor | Questions? Contact #dx-team or check out this page

@gitstream-cm

gitstream-cm Bot commented Jul 15, 2026

Copy link
Copy Markdown

🥷 Code experts: No results found

No code experts were identified for the files in this pull request based on git blame analysis.

This may occur when:

  • Files are new or have limited commit history
  • Git authors aren't mapped to current team members
  • Analysis thresholds need adjustment

If you expected to see expert suggestions, consider:

  • Reviewing your config.user_mapping settings

  • Adjusting the gt/lt parameters in your action

  • Verifying files have sufficient commit history

To learn more about /:\gitStream - Visit our Docs

@tf-seti tf-seti force-pushed the chore/remove-sonarcloud branch from 60cd46f to ffa65cf Compare July 15, 2026 12:43
@tf-seti tf-seti force-pushed the chore/remove-sonarcloud branch from ffa65cf to 2a5bf5c Compare July 15, 2026 13:06
@glean-github-app-oauth

Copy link
Copy Markdown

From Glean Code Writer

Updated the PR to remove the remaining SonarCloud reference (the comment # used in SonarCloud scan in the workflow file).

The PR now completely removes all SonarCloud code references:

  • ✅ Deleted sonar-project.properties
  • ✅ Removed the SonarCloud Scan step from .github/workflows/pull-request.yml
  • ✅ Removed the SonarCloud comment from the Node version matrix

Next steps (admin follow-ups):

  • Remove the SONAR_CLOUD_TOKEN repository secret
  • Archive the project in SonarCloud

@typeform-ops-gha

Copy link
Copy Markdown

[BOT] Preview available with hash 17990e9d930ee7ab9749a40a0660d82e041e4f4b here.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants