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
DateTimeCreateFromFormatCallFixer - Introduction #6286
DateTimeCreateFromFormatCallFixer - Introduction #6286
Conversation
f977829
to
90f4c4c
Compare
90f4c4c
to
5b42629
Compare
5b42629
to
e6c8569
Compare
@keradus Very helpful reviews, thanks. |
Please avoid force-pushes next time. Now, I have to review everything instead of checking the changes only |
dd9b46e
to
2090f0c
Compare
2090f0c
to
2e156d2
Compare
@liquid207 |
bbabf9f
to
bced21c
Compare
@SpacePossum |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks solid to me, thanks for developing this rule
One last little minor, can you squash into one commit with you as author and committer? It will make a nice single commit new feature and the PR will be marked as |
!
prefix to the first argument of DateTime::createFromFormatbced21c
to
2bad1a9
Compare
…:createFromFormat Co-authored-by: Dariusz Rumiński <dariusz.ruminski@gmail.com>
2bad1a9
to
1cda359
Compare
squashed, @keradus @SpacePossum Thanks for your help. |
Thank you @liquid207. |
thanks @liquid207 , nice one 👍 |
Closes #6133