Skip to content

Conversation

@stephane-monnot
Copy link
Contributor

No description provided.

@stephane-monnot
Copy link
Contributor Author

@tgalopin Sorry, I forgot some imports in my previous PR.

<?php

namespace FOS\MessageBundle\Search;
use FOS\MessageBundle\Model\ThreadInterface;
Copy link
Contributor

@tgalopin tgalopin Aug 26, 2016

Choose a reason for hiding this comment

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

Can you put an empty line before :) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

Choose a reason for hiding this comment

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

Not on this one :) . You had two.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. Other one ? :). My idea conf is a bit weird.

@tgalopin
Copy link
Contributor

Mergeable or WIP :) ?

@stephane-monnot
Copy link
Contributor Author

stephane-monnot commented Aug 26, 2016

Wait !!!

@tgalopin
Copy link
Contributor

I wait :) . When you are finished, can you squash into a single commit?

@stephane-monnot
Copy link
Contributor Author

Yes, thank you.

commit d89f441
Author: Monnot Stéphane <monnot.stephane@gmail.com>
Date:   Fri Aug 26 15:59:18 2016 +0200

    Add empty line between namespace and use lines

commit 9050a33
Author: Monnot Stéphane <monnot.stephane@gmail.com>
Date:   Fri Aug 26 15:52:37 2016 +0200

    Fix "collection of " doc blocks

commit 6089bb0
Author: Monnot Stéphane <monnot.stephane@gmail.com>
Date:   Fri Aug 26 15:32:12 2016 +0200

    Add empty line between namespace and use lines

commit 90556e3
Author: Monnot Stéphane <monnot.stephane@gmail.com>
Date:   Fri Aug 26 15:24:32 2016 +0200

    Add some imports for doc block

commit 342c31a
Author: Monnot Stéphane <monnot.stephane@gmail.com>
Date:   Fri Aug 26 14:47:22 2016 +0200

    Fix php docblock issues
@stephane-monnot stephane-monnot changed the title Add some imports for doc block Improve idea completion Aug 26, 2016
@stephane-monnot stephane-monnot force-pushed the feature/fixdocblockissues branch from d89f441 to 0592aa1 Compare August 26, 2016 14:29
@stephane-monnot
Copy link
Contributor Author

Mergeable and squashed.

@tgalopin tgalopin merged commit d33c18a into FriendsOfSymfony:master Aug 26, 2016
@tgalopin
Copy link
Contributor

Thanks!

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