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
PhpdocAnnotationWithoutDotFixer - Handle trailing whitespaces #3867
PhpdocAnnotationWithoutDotFixer - Handle trailing whitespaces #3867
Conversation
this looks awesome to me, thanks so much :) |
(on a side note, if this fixer is about changing text to a sentence without a stop maybe we should unify what we consider a stop (so not only dots), like done here) |
*/', | ||
'<?php | ||
/** | ||
* @throws \Exception having whitespaces after dot, yet I am fixed. '.' |
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.
Please add test case for trailing tab character too, as we are using \h
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.
ping @kubawerlos
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.
Sorry, I've missed that. Done.
Thank you @kubawerlos. |
…es (kubawerlos) This PR was squashed before being merged into the 2.12 branch (closes #3867). Discussion ---------- PhpdocAnnotationWithoutDotFixer - Handle trailing whitespaces Solves #3857 - I guess @SpacePossum will have some more fancy test cases. Commits ------- 3897460 PhpdocAnnotationWithoutDotFixer - Handle trailing whitespaces
Solves #3857 - I guess @SpacePossum will have some more fancy test cases.