Skip to content

[MOSIP-44319]Modify sonar analysis workflow configuration#158

Merged
mayuradesh merged 1 commit intoinji:developfrom
Mahesh-Binayak:patch-1
Feb 2, 2026
Merged

[MOSIP-44319]Modify sonar analysis workflow configuration#158
mayuradesh merged 1 commit intoinji:developfrom
Mahesh-Binayak:patch-1

Conversation

@Mahesh-Binayak
Copy link
Contributor

@Mahesh-Binayak Mahesh-Binayak commented Feb 2, 2026

Updated sonar analysis workflow to use a specific commit and changed secret key name.

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow configuration to use improved code analysis process with updated build parameters.

Updated sonar analysis workflow to use a specific commit and changed secret key name.

Signed-off-by: Mahesh-Binayak <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

Walkthrough

GitHub workflow configuration updated to reference a different branch of an external workflow file and rename an input parameter from SONAR_ORGANIZATION to ORG_KEY in the sonar analysis step.

Changes

Cohort / File(s) Summary
Workflow Configuration
.github/workflows/push-trigger.yml
Updated workflow reference branch from master-java21 to dsd9685 and renamed sonar input parameter SONAR_ORGANIZATION to ORG_KEY.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop and a skip, the workflows align,
Branch references dance in a new design,
Parameters renamed with clarity bright,
The sonar checks sparkle, everything's right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: modifying the sonar analysis workflow configuration, which matches the actual updates to the workflow file and parameter renaming.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mayuradesh mayuradesh merged commit 36ab226 into inji:develop Feb 2, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants