Skip to content

MX-227: Use ANSI SQL in the Liquibase Scripts#105

Merged
IOhacker merged 1 commit intoopenMF:developfrom
IOhacker:develop
Apr 8, 2026
Merged

MX-227: Use ANSI SQL in the Liquibase Scripts#105
IOhacker merged 1 commit intoopenMF:developfrom
IOhacker:develop

Conversation

@IOhacker
Copy link
Copy Markdown
Contributor

@IOhacker IOhacker commented Apr 8, 2026

Summary by CodeRabbit

  • Chores
    • Updated license header and copyright notice from Apache License 2.0 to Mozilla Public License v2.0.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8e108348-6a97-485d-ba52-e1664a08ea98

📥 Commits

Reviewing files that changed from the base of the PR and between acc7dfa and d4c00ab.

📒 Files selected for processing (1)
  • src/main/resources/db/changelog/tenant/module/selfservice/module-changelog-master.xml

📝 Walkthrough

Walkthrough

Updated the file header license from Apache License 2.0 to Mozilla Public License v2.0, with copyright attribution changed to "Mifos Initiative" effective 2026. No functional code changes.

Changes

Cohort / File(s) Summary
License Header Update
src/main/resources/db/changelog/tenant/module/selfservice/module-changelog-master.xml
Replaced Apache Software Foundation license header with Mozilla Public License v2.0 and updated copyright attribution.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

⏱️ <10 Min Review

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title references using ANSI SQL in Liquibase scripts, but the actual change only modifies the license header in an XML changelog file with no changes to SQL content or Liquibase script functionality. Revise the title to accurately reflect the change, such as 'Update license header in changelog master XML file' or provide SQL changes that align with the stated objective.
✅ 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

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.

@IOhacker IOhacker merged commit 2dcc4a0 into openMF:develop Apr 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant