Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[5.0][Finder] add parameter type-hints #32243

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
Jul 8, 2019

Conversation

smoench
Copy link
Contributor

@smoench smoench commented Jun 28, 2019

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #32179
License MIT
Doc PR N/A

This PR adds parameter type hints to the Finder component.

@smoench smoench force-pushed the finder-parameter-type-hints branch 2 times, most recently from 0046e9c to 79f2f2e Compare June 28, 2019 07:36
@xabbuh xabbuh added this to the 5.0 milestone Jun 28, 2019
src/Symfony/Component/Finder/Finder.php Show resolved Hide resolved
src/Symfony/Component/Finder/Iterator/SortableIterator.php Outdated Show resolved Hide resolved
@derrabus derrabus mentioned this pull request Jun 28, 2019
57 tasks
@smoench smoench force-pushed the finder-parameter-type-hints branch from 79f2f2e to 1f6c68c Compare June 28, 2019 09:28
nicolas-grekas added a commit that referenced this pull request Jun 28, 2019
This PR was merged into the 3.4 branch.

Discussion
----------

[Finder] docblock fixes

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | N/A
| License       | MIT
| Doc PR        | N/A

This PR provides some docblock fixes found in #32243

Commits
-------

02ee4d0 [Finder] docblock fixes
@smoench smoench force-pushed the finder-parameter-type-hints branch from 1f6c68c to 7f98903 Compare July 5, 2019 07:03
@fabpot
Copy link
Member

fabpot commented Jul 8, 2019

Thank you @smoench.

@fabpot fabpot merged commit 7f98903 into symfony:master Jul 8, 2019
fabpot added a commit that referenced this pull request Jul 8, 2019
This PR was merged into the 5.0-dev branch.

Discussion
----------

[5.0][Finder] add parameter type-hints

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #32179
| License       | MIT
| Doc PR        | N/A

This PR adds parameter type hints to the Finder component.

Commits
-------

7f98903 [5.0][Finder] add parameter type-hints
@smoench smoench deleted the finder-parameter-type-hints branch July 8, 2019 06:42
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.

5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.