Skip to content

Update push-trigger.yml#620

Merged
swatigoel merged 1 commit intoinji:release-0.14.xfrom
rajapandi1234:patch-2
Feb 18, 2026
Merged

Update push-trigger.yml#620
swatigoel merged 1 commit intoinji:release-0.14.xfrom
rajapandi1234:patch-2

Conversation

@rajapandi1234
Copy link
Contributor

@rajapandi1234 rajapandi1234 commented Feb 18, 2026

Summary by CodeRabbit

  • Chores
    • Updated continuous integration workflow configuration for code analysis and team notifications.

Signed-off-by: Rajapandi M <138785181+rajapandi1234@users.noreply.github.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 18, 2026

Walkthrough

The workflow configuration in .github/workflows/push-trigger.yml has been updated to reference a different workflow revision for the sonar analysis job and to use a different Slack webhook secret for notifications.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/push-trigger.yml
Updated sonar analysis workflow reference from master-java21 to dsd9685, and changed Slack notification secret from SLACK_WEBHOOK to SLACK_WEBHOOK_INJI_TEAM.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A workflow takes flight with paths anew,
Secrets whisper their names in gentle hue,
Sonar dances to a different beat—dsd's tune,
Slack bells chime for Inji's crew under the moon! 🌙

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and generic, using a non-descriptive term that doesn't convey meaningful information about the specific changes made. Use a more specific title that describes the actual changes, such as 'Update sonar analysis workflow reference and Slack webhook secret' or 'Update workflow dependencies and notification configuration'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/push-trigger.yml:
- Line 172: The `sonar_analysis` job's action reference is using the dev branch
suffix `@dsd9685`; update the `uses:
mosip/kattu/.github/workflows/maven-sonar-analysis.yml@dsd9685` line to use
`@master-java21` so the `sonar_analysis` job matches the other jobs' stable
branch (`master-java21`) for consistency and stability.

@swatigoel swatigoel merged commit 504cb88 into inji:release-0.14.x Feb 18, 2026
14 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

Comments