Skip to content

Conversation

@ckm007
Copy link
Contributor

@ckm007 ckm007 commented Jan 8, 2026

Summary by CodeRabbit

  • Chores
    • Updated internal build pipeline configurations to use specific workflow versions for artifact publishing.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: Chandra Keshav Mishra <chandrakeshavmishra@gmail.com>
@coderabbitai
Copy link

coderabbitai bot commented Jan 8, 2026

Walkthrough

Two GitHub Actions workflow references in kotlin-artifacts-build.yml are updated from master to a specific branch/commit MOSIP-44195. The changes apply to both snapshot and release publishing jobs, affecting only the workflow source references while maintaining all other inputs and steps.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/kotlin-artifacts-build.yml
Updated workflow references for both publish-snapshot and publish-release jobs from master branch to MOSIP-44195 commit/branch reference (+2/-2)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Prafulrakhade

Poem

🐰 Workflows now point to MOSIP's way,
No longer lost in master's bay,
With branches fresh and commits true,
Our artifacts build clean and new!

🚥 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 references the JIRA ticket MOSIP-44195 and accurately describes the main change: updating kotlin-artifacts-build.yml to reference a specific commit/branch instead of master.
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.


📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eff9f4a and 11cba8d.

📒 Files selected for processing (1)
  • .github/workflows/kotlin-artifacts-build.yml
🔇 Additional comments (2)
.github/workflows/kotlin-artifacts-build.yml (2)

45-45: Remove concern about workflow file verification—the file exists and is accessible at the branch reference.

Both the publish-snapshot (line 9) and publish-release (line 26) jobs consistently use @MOSIP-44195, a branch reference in the mosip/kattu repository. The maven-publish-android-download.yaml workflow file is confirmed to exist at this reference. If your team intends to use a stable tag instead of a branch reference for long-term reliability, consider updating to a tag; otherwise, this reference is valid and functional.


28-28: Remove this review comment; the file was created with @MOSIP-44195, not changed from @master.

The review comment assumes this is a change from @master to @MOSIP-44195, but the git history shows .github/workflows/kotlin-artifacts-build.yml was newly created in commit 11cba8d with @MOSIP-44195 as the initial reference. There is no prior version using @master to "switch" from. The comment's premise is incorrect.

Likely an incorrect or invalid review 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.

@Prafulrakhade Prafulrakhade merged commit e12d82b into develop Jan 8, 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.

3 participants