Skip to content

Conversation

@PATROMO
Copy link

@PATROMO PATROMO commented Jan 16, 2013

corrected the namespaces.

corrected the namespaces.
@franzliedke
Copy link
Contributor

Looks like you broke the line endings, though.

@jasonlewis
Copy link
Contributor

There's no reason to prefix it with the backslash is there?

@franzliedke
Copy link
Contributor

Jason, that was discussed in another issue. Basically, it's about IDE support. Taylor essentially said something along the lines of he might change them to this "proper" way while changing other things, but I'm sure he's thankful for somebody doing all this work for him.

@jasonlewis
Copy link
Contributor

Ah okay then, fair enough! :)

@bencorlett
Copy link
Contributor

Looks good :)

@Sopamo
Copy link
Contributor

Sopamo commented Jan 29, 2013

👍 It is really annoying to not have autocompletion. Somehow makes the complete PHPDocs useless.

joelharkes pushed a commit to joelharkes/framework_old that referenced this pull request Mar 7, 2019
…n-endpoints-tests

Subscription endpoints tests
gonzalom pushed a commit to Hydrane/tmp-laravel-framework that referenced this pull request Oct 12, 2023
Type hint parameter to match the changes in contract
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.

6 participants