ci(sonar,netlify): use SonarCloud Automatic Analysis and disable Netlify PR previews - #149
Conversation
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
✅ Deploy Preview for adt-cli canceled.
|
|
View your CI Pipeline Execution ↗ for commit da20926
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 25 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Update scripts/sonar-monorepo.ts so the root Nx project (abapify) gets the remaining top-level directories that are not Nx project roots. This keeps per-package scans separate while scanning the rest of the repository (src, .github, scripts, docs, openspec, gascity, etc.) in a dedicated monorepo root project without overlapping sources. Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
|
Skipping PR review because a bot author is detected. If you want to trigger CodeAnt AI, comment |
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
980cf78 to
0609905
Compare
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 25 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
- Remove the CI-based Sonar matrix workflow, matrix files, and generator script now that SonarCloud Automatic Analysis is enabled. - Replace sonar-project.properties with .sonarcloud.properties scoped to the whole repository and excluding generated/build/vendored directories. - Clean obsolete Codacy exclusions for the removed Sonar files. Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
devin-ai-integration
Bot
changed the title
Up to standards ✅🟢 Issues
|
… minutes Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
… devin/sonar-monorepo-setup
devin-ai-integration
Bot
changed the title
|
1 participant
Summary
Switch the repository to SonarQube Cloud Automatic Analysis (GitHub App) for the whole repo, and stop Netlify from running deploy previews and branch deploys on pull requests to save build minutes. Production deploys from
mainstill run.What changed:
.github/workflows/sonar.yml,scripts/sonar-monorepo.ts,sonar-matrix.json,sonar-monorepo.json).sonar-project.propertieswith.sonarcloud.propertiesso Automatic Analysis scopessonar.sourcesto.and excludes generated/build/vendored directories..codacy.ymlexclusions for the removed Sonar files.[context.deploy-preview]and[context.branch-deploy]ignore = "exit 0"tonetlify.tomlso Netlify only builds/deploys the production context frommain.Notes:
abapify_adt-cli_*SonarCloud projects were removed and only the existingabapify_adt-cliproject remains bound to this repository.sonar.issue.ignore.*rules are not honored by Automatic Analysis; any false-positive suppressions must be configured in the SonarCloud UI under Administration > General Settings > Analysis Scope > Ignore Issues on Multiple Criteria.Link to Devin session: https://app.devin.ai/sessions/717d08150ac64c689c436d28aa11182c
Requested by: @ThePlenkov