Skip to content

Add soft @return type declarations to interfaces and base classes#44

Open
mpdude wants to merge 1 commit into
FriendsOfBehat:masterfrom
mpdude:interface-strict-types
Open

Add soft @return type declarations to interfaces and base classes#44
mpdude wants to merge 1 commit into
FriendsOfBehat:masterfrom
mpdude:interface-strict-types

Conversation

@mpdude

@mpdude mpdude commented Dec 17, 2025

Copy link
Copy Markdown

Announcing the return types this way may allow us to use language-level type declarations in the next major version.

mpdude added a commit to mpdude/MinkExtension that referenced this pull request Dec 17, 2025
Squash-merge of FriendsOfBehat#44

commit b9b8be4
Author: Matthias Pigulla <mp@webfactory.de>
Date:   Wed Dec 17 18:44:01 2025 +0100

    Add soft `@return` type declarations to interfaces and base classes
@mpdude mpdude force-pushed the interface-strict-types branch from b9b8be4 to bf8363d Compare December 17, 2025 18:24
Comment thread CHANGELOG.md Outdated
@mpdude mpdude force-pushed the interface-strict-types branch from 7813724 to e8ae83a Compare December 17, 2025 20:38
@mpdude mpdude force-pushed the interface-strict-types branch from e8ae83a to bdbdc2c Compare December 19, 2025 17:37
@mpdude

mpdude commented Dec 19, 2025

Copy link
Copy Markdown
Author

Rebased to resolve merge conflicts

@mpdude mpdude force-pushed the interface-strict-types branch from bdbdc2c to c92dbb0 Compare January 14, 2026 18:21
@mpdude mpdude requested a review from stof January 14, 2026 18:22
@mpdude

mpdude commented Jun 24, 2026

Copy link
Copy Markdown
Author

@pamil at least as of #51, master contains breaking changes that will need to land in a new major version (dropping GoutteFactory, adding return types).

This PR here was intended to prepare those changes in a non-breaking way and to advise downstream users of upcoming changes.

If you are still interested in this, please adivse – we'd probably at least need a 2.x branch where we can merge this, and then merge-up into master or 3.x for the upcoming next major.

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.

3 participants