Skip to content

[Finder] Restore Finder:getIterator return type PHPDoc #44731

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 21, 2021
Merged

[Finder] Restore Finder:getIterator return type PHPDoc #44731

merged 1 commit into from
Dec 21, 2021

Conversation

kubawerlos
Copy link
Contributor

Q A
Branch? 6.0
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #44697
License MIT

As discussed in the related ticket we want this annotation back. Targeting 6.0 as it is present in 5.4, was remove during merging 5.4 up to 6.0.

@carsonbot
Copy link

It looks like you unchecked the "Allow edits from maintainer" box. That is fine, but please note that if you have multiple commits, you'll need to squash your commits into one before this can be merged. Or, you can check the "Allow edits from maintainers" box and the maintainer can squash for you.

Cheers!

Carsonbot

@kubawerlos
Copy link
Contributor Author

No dear bot @carsonbot, this one is a GitHub issue: community/community#5634

@stof
Copy link
Member

stof commented Dec 20, 2021

oh, I was not aware of the fact that this feature was not available for forks belonging to an organization. Maybe carsonbot could detect that case.

@fancyweb fancyweb changed the base branch from 6.1 to 6.0 December 20, 2021 17:48
@fancyweb fancyweb modified the milestones: 6.1, 6.0 Dec 20, 2021
@carsonbot carsonbot changed the title Restore Finder:getIterator return type PHPDoc [Finder] Restore Finder:getIterator return type PHPDoc Dec 20, 2021
@kubawerlos
Copy link
Contributor Author

@stof or GitHub will fix this, I didn't know about this as well, until carsonbot's message, doesn't look like a feature.

@derrabus
Copy link
Member

Good catch, thanks @kubawerlos.

@derrabus derrabus merged commit 4d164da into symfony:6.0 Dec 21, 2021
fabpot added a commit that referenced this pull request Dec 21, 2021
…ovariant IteratorAggregate implementations (derrabus)

This PR was merged into the 6.0 branch.

Discussion
----------

[EventDispatcher][HttpFoundation] Restore return type to covariant IteratorAggregate implementations

| Q             | A
| ------------- | ---
| Branch?       | 6.0
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Follows #44731
| License       | MIT
| Doc PR        | N/A

Commits
-------

0f2b5bb Restore return type to covariant IteratorAggregate implementations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Symfony\Component\Finder\Finder::getIterator return type annotation removed
5 participants