Skip to content

Refactored plugin system logs + Plugin version resolver - #57

Merged
nil-malh merged 2 commits into
mainfrom
refactor/plugin-system
Aug 10, 2026
Merged

Refactored plugin system logs + Plugin version resolver #57
nil-malh merged 2 commits into
mainfrom
refactor/plugin-system

Conversation

@nil-malh

@nil-malh nil-malh commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

📝 Description


🔄 Type of Change

  • ♻️ Refactoring / technical debt — no functional change

✅ Pre-Merge Checklist

Code Quality

  • My code follows the Palantir Java Format style (passes mvn spotless:check)
  • I have added or updated Javadoc on public types and methods

Tests

  • I have added unit tests that cover the new / changed behaviour
  • All existing and new tests pass locally (mvn verify)
  • JaCoCo line coverage ≥ 70 % is maintained
  • Tests using Testcontainers are isolated and do not leave containers running

Licensing

  • Apache 2.0 license header has been added to every new source file (using spotless)

🧪 How to Test Locally

# 1. Build and run all tests (requires Docker for Testcontainers)
mvn verify

# 2. Check code style
mvn spotless:check

📸 Screenshots / Logs


📚 Additional Context

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

📋 Unreleased Changelog Preview

This is what the next release notes will look like based on commits in this PR.

Changelog

All notable changes to this project will be documented in this file.

[Unreleased]

♻️ Refactoring

  • Improve logging messages for PluginSystem and add PluginVersionResolver class

  • Refactored plugin init logs


Generated by git-cliff


🔄 Run #142 · Sun, 09 Aug 2026 17:41:08 GMT

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

✅ Test Results

Metric Count
Passed 720
Failed 0
⏭️ Skipped 0
📊 Total 720

✅ Coverage

Type Coverage Covered / Total
📏 Lines 83.8% 1877 / 2240
🌿 Branches 72.9% 583 / 800
🔧 Methods 84.7% 387 / 457

🔄 CI run #161 · Sun, 09 Aug 2026 17:44:43 GMT

@nil-malh
nil-malh merged commit 5beb726 into main Aug 10, 2026
10 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.

1 participant