Skip to content

[PSR-4 Split] Split interfaces and traits as well#1414

Merged
TomasVotruba merged 1 commit intomasterfrom
split-all-classlike-psr4
May 12, 2019
Merged

[PSR-4 Split] Split interfaces and traits as well#1414
TomasVotruba merged 1 commit intomasterfrom
split-all-classlike-psr4

Conversation

@JanMikes
Copy link
Copy Markdown
Contributor

No description provided.

@JanMikes JanMikes requested a review from TomasVotruba May 10, 2019 21:27
@JanMikes
Copy link
Copy Markdown
Contributor Author

@TomasVotruba before this MR files containing interface and/or trait with class would fail splitting - it deleted non class nodes and did not move them to new files.

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented May 10, 2019

Better create new test instead of deleting the old one.

It's hard to read PR now

@JanMikes
Copy link
Copy Markdown
Contributor Author

Fixed 😉

@JanMikes JanMikes force-pushed the split-all-classlike-psr4 branch from c4cf354 to 31d2ce4 Compare May 10, 2019 22:29
@JanMikes
Copy link
Copy Markdown
Contributor Author

@TomasVotruba can we merge? 😉

@TomasVotruba
Copy link
Copy Markdown
Member

TomasVotruba commented May 12, 2019

Sure, I didn't have time to review. All the Italian 🍕 🍷 and travelling :D

Nice test btw, simple and clear 👍

@TomasVotruba TomasVotruba merged commit de4cca7 into master May 12, 2019
@TomasVotruba TomasVotruba deleted the split-all-classlike-psr4 branch May 12, 2019 16:45
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