Skip to content

fix: remove reference to removed deployer-instance package#15

Merged
konradmichalik merged 1 commit intomainfrom
fix/remove-deployer-instance-reference
Feb 18, 2026
Merged

fix: remove reference to removed deployer-instance package#15
konradmichalik merged 1 commit intomainfrom
fix/remove-deployer-instance-reference

Conversation

@konradmichalik
Copy link
Contributor

@konradmichalik konradmichalik commented Feb 18, 2026

Summary

  • Remove obsolete sourcebroker/deployer-instance path from TYPO3 and Symfony Loaders
  • Package was removed as dependency in deployer-extended v20.0.0, causing require_once failures

Changes

  • deployer/typo3/Loader.php - Remove deployer-instance path from Load config
  • deployer/symfony/Loader.php - Remove deployer-instance path from Load config

Summary by CodeRabbit

  • Chores
    • Removed redundant vendor path references from deployer loader configurations in Symfony and TYPO3, streamlining the deployment initialization process.

@coderabbitai
Copy link

coderabbitai bot commented Feb 18, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Removed the vendor/sourcebroker/deployer-instance/deployer path from the Load constructor initialization array in two Symfony and TYPO3 Loader files. The vendor/sourcebroker/deployer-extended/deployer path remains. No logic or control flow changes.

Changes

Cohort / File(s) Summary
Loader initialization simplification
deployer/symfony/Loader.php, deployer/typo3/Loader.php
Removed the vendor/sourcebroker/deployer-instance/deployer entry from the Load constructor array; only vendor/sourcebroker/deployer-extended/deployer path is now loaded.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A line removed with care,
Two loaders now much spare,
Extended paths remain so true,
Instance paths bid adieu! ✨

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/remove-deployer-instance-reference

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.

@konradmichalik konradmichalik merged commit f5569b1 into main Feb 18, 2026
1 of 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.

1 participant

Comments