Skip to content
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

[6.x] Sort missed stub imports #29958

Merged
merged 1 commit into from
Sep 11, 2019
Merged

Conversation

netpok
Copy link
Contributor

@netpok netpok commented Sep 11, 2019

Previous pull request #29954 missed some import, this pull request fixes those.

This is a cosmetic change that does not affect the output in any way as it will be sorted after stub compiling.

This pull request was generated with the same code used after compiling and run on every *.stub so there shouldn't be any missed stub.

@taylorotwell taylorotwell merged commit e879699 into laravel:6.x Sep 11, 2019
@netpok netpok deleted the feature/sort-imports branch September 11, 2019 21:25
@antonkomarev
Copy link
Contributor

They were in this state because without your PR #29951 they all will have App name by default and were meant to be on the first place to be in right order. But your addition of import sorter let us to keep style consistency even in stub files. 💟

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.

3 participants