Skip to content

Add support to prefixed containers - #2

Merged
dpanta94 merged 1 commit into
dpanta94:mainfrom
estevao90:configurable-container
Jun 8, 2026
Merged

Add support to prefixed containers#2
dpanta94 merged 1 commit into
dpanta94:mainfrom
estevao90:configurable-container

Conversation

@estevao90

Copy link
Copy Markdown
Contributor

Summary

This pull request adds support for namespace-prefixed (e.g., Strauss/Mozart) container interfaces in PHPStan by introducing a configurable extension. This allows dynamic return type resolution for containers whose interfaces have been rewritten and no longer share ancestry with the bundled StellarWP or PSR interfaces.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • ⚡ Performance improvement
  • ♻️ Code refactoring
  • 🧪 Test improvements
  • 🔧 Chore (build process, dependencies, etc.)

Related Issues

Closes #

Pre-Submission Checklist

Code Quality

  • PHPStan analysis passes
  • Coding standards pass
  • All tests pass
  • No debug code (var_dump, error_log, etc.) left in production code
  • No commented-out code unless specifically needed for reference
  • Documentation updated for any new features or changed behavior
  • New tests have been added for new functionality
  • All existing tests continue to pass

📖 Read the full contributing guidelines: CONTRIBUTING.md

@estevao90
estevao90 marked this pull request as ready for review June 4, 2026 13:44

@dpanta94 dpanta94 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you mate!

@dpanta94
dpanta94 merged commit 1fe2467 into dpanta94:main Jun 8, 2026
7 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