Skip to content

fix: add sonar and change the dependabot.yaml file#90

Merged
nnicora merged 1 commit into
mainfrom
fix/add-sonar-adn-dependbot
Mar 12, 2026
Merged

fix: add sonar and change the dependabot.yaml file#90
nnicora merged 1 commit into
mainfrom
fix/add-sonar-adn-dependbot

Conversation

@nnicora
Copy link
Copy Markdown
Contributor

@nnicora nnicora commented Mar 12, 2026

Add SonarQube Analysis Workflow and Update Dependabot Configuration

Chore

🔧 Added a SonarQube analysis workflow and updated the Dependabot configuration to improve update scheduling and grouping across all package ecosystems.

Changes

  • .github/workflows/sonar.yaml: Added a new secure SonarQube analysis workflow that triggers on completion of the Quality workflow. It reuses a shared workflow from openkcm/build with inherited secrets.

  • .github/dependabot.yaml: Updated all three package ecosystems (gomod, docker, github-actions) with the following changes:

    • Changed update interval from daily to weekly for all ecosystems.
    • Added commit-message configuration with fix prefix and scope.
    • Added dependency grouping for minor and patch version updates.
    • Added open-pull-requests-limit: 5 for the docker and github-actions ecosystems.
    • Fixed indentation issues throughout the file.
  • 🔄 Regenerate and Update Summary

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 12, 2026

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMar 12, 2026, 11:05 AM

@push-tags-from-workflow push-tags-from-workflow Bot added bug Something isn't working ci-cd labels Mar 12, 2026
Copy link
Copy Markdown

@hyperspace-insights hyperspace-insights Bot left a comment

Choose a reason for hiding this comment

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

The PR is straightforward and well-structured — the Sonar workflow is correctly wired to trigger on the existing "Quality" workflow, and the dependabot grouping/scheduling changes are consistent across all three ecosystems. The only minor issues flagged are missing trailing newlines at the end of both modified/added files.

PR Bot Information

Version: 1.18.4 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • Correlation ID: 5f3b4110-1e03-11f1-9a2c-68268457f7ce
  • LLM: anthropic--claude-4.6-sonnet
  • Event Trigger: pull_request.opened

jobs:
check:
uses: openkcm/build/.github/workflows/sonar.yaml@main
secrets: inherit No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Best Practices: File is missing a newline at end of file

The file ends without a trailing newline. POSIX-compliant text files should end with a newline; many tools (git diff, linters) will warn about files that lack one. Consider adding a trailing newline after the last line.


Please provide feedback on the review comment by checking the appropriate box:

  • 🌟 Awesome comment, a human might have missed that.
  • ✅ Helpful comment
  • 🤷 Neutral
  • ❌ This comment is not helpful

Comment thread .github/dependabot.yaml
update-types:
- "minor"
- "patch"
open-pull-requests-limit: 5 No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Best Practices: File is missing a newline at end of file

The file ends without a trailing newline. POSIX-compliant text files should end with a newline; many tools (git diff, linters) will warn about files that lack one. Consider adding a trailing newline after the last line.


Please provide feedback on the review comment by checking the appropriate box:

  • 🌟 Awesome comment, a human might have missed that.
  • ✅ Helpful comment
  • 🤷 Neutral
  • ❌ This comment is not helpful

@nnicora nnicora merged commit 46bcc04 into main Mar 12, 2026
7 checks passed
@nnicora nnicora deleted the fix/add-sonar-adn-dependbot branch March 12, 2026 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci-cd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant