Skip to content

[PostRector] Remove CurrentFileProvider on PostFileProcessor#5097

Merged
TomasVotruba merged 3 commits intomainfrom
post-file
Oct 1, 2023
Merged

[PostRector] Remove CurrentFileProvider on PostFileProcessor#5097
TomasVotruba merged 3 commits intomainfrom
post-file

Conversation

@samsonasik
Copy link
Copy Markdown
Member

Directly pass file name from FileProcessor.

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba it is ready for review.

@samsonasik
Copy link
Copy Markdown
Member Author

This also reduce repetitive $file->getFilePath() 👍

@samsonasik
Copy link
Copy Markdown
Member Author

All checks have passed 🎉 @TomasVotruba I think it is ready.

@TomasVotruba
Copy link
Copy Markdown
Member

Looks good 👏

@TomasVotruba TomasVotruba merged commit 8adfc7a into main Oct 1, 2023
@TomasVotruba TomasVotruba deleted the post-file branch October 1, 2023 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants