Skip to content

[DSD-9403] mosipdev to mosipdev2 - #1433

Merged
abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
SwapnilWorks-Devops:release-1.2.0.1
Dec 3, 2025
Merged

[DSD-9403] mosipdev to mosipdev2#1433
abhishek8shankar merged 1 commit into
mosip:release-1.2.0.1from
SwapnilWorks-Devops:release-1.2.0.1

Conversation

@SwapnilWorks-Devops

@SwapnilWorks-Devops SwapnilWorks-Devops commented Dec 3, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated service image configurations by removing entries for five services and adding entries for two new services to support updated deployment requirements.

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

Signed-off-by: Swapnil <swapnil.mohanty@technoforte.co.in>
@coderabbitai

coderabbitai Bot commented Dec 3, 2025

Copy link
Copy Markdown

Walkthrough

This PR updates the vidivi images manifest by removing five image entries (signup-with-plugins, esignet-with-plugins, mock-relying-party-ui, mock-relying-party-service, mock-identity-system) and adding two new entries (inji-web and mimoto, both from the develop branch).

Changes

Cohort / File(s) Change Summary
Image manifest configuration
release/vidivi/images.txt
Removed 5 image entries (signup-with-plugins, esignet-with-plugins, mock-relying-party-ui, mock-relying-party-service, mock-identity-system); added 2 new image entries (inji-web:develop, mimoto:develop)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Verify that the removed image entries are no longer needed
  • Confirm that inji-web and mimoto are correct image names with appropriate develop branch tags

Possibly related PRs

Suggested reviewers

  • ckm007

Poem

🐰 Five old images fade away,
Two fresh ones join the fray,
Inji-web and mimoto bloom,
Clearing cluttered manifest room! 🎉

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title 'DSD-9403 mosipdev to mosipdev2' does not accurately reflect the actual changes, which involve removing five MOSIP image entries and adding two new inji-web and mimoto entries. Revise the title to clearly describe the actual changes, such as 'Update image entries: remove release branches and add develop branches' or 'Replace legacy image entries with inji-web and mimoto develop versions'.
✅ 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.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dc0a2c9 and f3b42c3.

📒 Files selected for processing (1)
  • release/vidivi/images.txt (1 hunks)
🔇 Additional comments (1)
release/vidivi/images.txt (1)

1-2: Both flagged concerns do not appear in the current file state. The file contains only five entries, all using mosipdev/ registry prefix with release-specific tags (:release-1.3.x, :release-1.7.x, :release-0.12.x). No mosipdev2/ references or :develop tags are present. The two lines shown in the annotation (inji-web:develop and mimoto:develop) do not exist in the file.

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.

@abhishek8shankar
abhishek8shankar merged commit 85e1406 into mosip:release-1.2.0.1 Dec 3, 2025
2 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