Skip to content

[FrameworkBundle] fix search in debug autowiring #30887

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

Conversation

Simperfit
Copy link
Contributor

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

Taking #30522 and finishing it with @nicolas-grekas comments.
Is the sentence ok ?

@Simperfit Simperfit force-pushed the feature/fix-search-in-debug-autowiring branch 2 times, most recently from cff1c8e to bfe246c Compare April 6, 2019 12:39
@Simperfit
Copy link
Contributor Author

Simperfit commented Apr 6, 2019

@chalasr @fabpot PR is ready, the message has been changed.

@Simperfit Simperfit force-pushed the feature/fix-search-in-debug-autowiring branch 2 times, most recently from cbec3df to 92155ec Compare April 6, 2019 15:23
@Simperfit
Copy link
Contributor Author

@nicolas-grekas updated

@soufianZantar
Copy link
Contributor

soufianZantar commented Apr 6, 2019

I spent my time in debbuging and crating my PR, and i waiting for the back from 24days, @Simperfit it's normal for you to create new PR after the back of @nicolas-grekas ????

@chalasr it's normal to close my PR and not this PR?

@Simperfit Simperfit force-pushed the feature/fix-search-in-debug-autowiring branch from 92155ec to c965a17 Compare April 6, 2019 15:49
@Simperfit
Copy link
Contributor Author

@soufianZantar Firstly Thanks very much for the contribution to the Symfony framework, We are at a hackathon, we wanted to move things forward by taking the stalling PRs to finish them.

You still have the work done with the commit included in the PR, again thanks very much, I just took what you have done and updated it while talking to @nicolas-grekas.

@Simperfit Simperfit force-pushed the feature/fix-search-in-debug-autowiring branch from c965a17 to a535bdb Compare April 6, 2019 16:31
@soufianZantar
Copy link
Contributor

Ok, I understood now. No problem.

@greg0ire
Copy link
Contributor

greg0ire commented Apr 6, 2019

If you want to get credit for that PR, you can do so by associating the email address for what seems to be an alternate account (@sez-open ?) to your @soufianZantar account. With love from the hackathon: https://twitter.com/hashtag/EUFOSSA

@Simperfit Simperfit force-pushed the feature/fix-search-in-debug-autowiring branch from a535bdb to fec4bea Compare April 7, 2019 07:27
@nicolas-grekas
Copy link
Member

Thank you @Simperfit.

@nicolas-grekas nicolas-grekas merged commit fec4bea into symfony:master Apr 7, 2019
nicolas-grekas added a commit that referenced this pull request Apr 7, 2019
…open)

This PR was merged into the 4.3-dev branch.

Discussion
----------

[FrameworkBundle] fix search in debug autowiring

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #30493   <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        |

<!--
Write a short README entry for your feature/bugfix here (replace this comment block.)
This will help people understand your PR and can be used as a start of the Doc PR.
Additionally:
 - Bug fixes must be submitted against the lowest branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the master branch.
-->

Taking #30522 and finishing it with @nicolas-grekas comments.
Is the sentence ok ?

Commits
-------

fec4bea fix debug:autowiringcommand
@nicolas-grekas
Copy link
Member

And thank you @soufianZantar

@Simperfit Simperfit deleted the feature/fix-search-in-debug-autowiring branch April 7, 2019 12:12
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.3 Apr 30, 2019
@fabpot fabpot mentioned this pull request May 9, 2019
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.

8 participants