Clean up hook

[fogelito]

Summary by CodeRabbit

Release Notes

• New Features
  • Extended migration lifecycle with an additional cleanup stage, providing more granular control over migration operations.

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

[coderabbitai]

Walkthrough

A new public method cleanUp(): void is added to the Target class in src/Migration/Target.php. The method is currently a no-op placeholder with accompanying documentation. No existing methods are modified, and no behavioral changes are introduced. The addition expands the public API of the Target class to include this new lifecycle hook.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks and finishing touches

✅ 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 'Clean up hook' accurately describes the main change: adding a new cleanUp() lifecycle hook method to the Target class.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

src/Migration/Target.php (1)

243-248: Implementation is correct; consider enhancing the docblock for clarity.

The implementation follows the established pattern for lifecycle hooks in this class (alongside shutdown(), success(), and error()). However, the docblock could be more descriptive to help developers understand the purpose and timing of this hook.

💡 Optional: Enhanced documentation example

     /**
-     * Clean up callback
+     * Clean up callback
+     *
+     * This hook is called to perform cleanup operations after migration completion.
+     * Override this method in subclasses to implement custom cleanup logic
+     * (e.g., closing connections, releasing resources, temporary file removal).
      */
     public function cleanUp(): void

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a4b99c3 and 3236527.

📒 Files selected for processing (1)

• src/Migration/Target.php