8000 Update PHPDocs for IDE support. by PATROMO · Pull Request #72 · laravel/framework · GitHub
[go: up one dir, main page]
Skip to content

Update PHPDocs for IDE support. #72

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 5 commits into from
Closed

Update PHPDocs for IDE support. #72

wants to merge 5 commits into from

Conversation

PATROMO
Copy link
@PATROMO PATROMO commented Jan 16, 2013

corrected the namespaces.

@PATROMO
Update PHPDocs
22f26ff 
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.

@jonphipps jonphipps mentioned this pull request Feb 27, 2013
Open
joelharkes pushed a commit to joelharkes/framework_old that referenced this pull request Mar 7, 2019
@jaureguivictoria
Merge pull request laravel#72 from eurides-eu/feature/add-subscriptio…
e419a6e 
…n-endpoints-tests

Subscription endpoints tests
@WarcraftLogs WarcraftLogs mentioned this pull request Nov 22, 2020
Closed
gonzalom pushed a commit to Hydrane/tmp-laravel-framework that referenced this pull request Oct 12, 2023
@taylorotwell
Merge pull request laravel#72 from lukasgeiter/make-type-hint
611751e 
Type hint parameter to match the changes in contract
@mantas-done mantas-done mentioned this pull request May 14, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@PATROMO @franzliedke @jasonlewis @bencorlett @Sopamo @taylorotwell
0