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

DX: PhpdocAlignFixer - refactor to use DocBlock #4205

Merged
merged 1 commit into from Jan 4, 2019
Merged

DX: PhpdocAlignFixer - refactor to use DocBlock #4205

merged 1 commit into from Jan 4, 2019

Conversation

kubawerlos
Copy link
Contributor

As the change itself - using DocBlock instead of reinventing the wheel way - feels right there is also a surprise when this will be merged ;)

This is last one for today - Happy New Year to everybody! Long live the open source!

@keradus keradus added this to the 2.12.6 milestone Jan 4, 2019
@keradus
Copy link
Member

keradus commented Jan 4, 2019

Thank you @kubawerlos.

@keradus keradus merged commit 9b98c63 into PHP-CS-Fixer:2.12 Jan 4, 2019
keradus added a commit that referenced this pull request Jan 4, 2019
This PR was squashed before being merged into the 2.12 branch (closes #4205).

Discussion
----------

DX: PhpdocAlignFixer - refactor to use DocBlock

As the change itself - using `DocBlock` instead of reinventing the wheel way - feels right there is also a surprise when this will be merged ;)

This is last one for today - Happy New Year to everybody! Long live the open source!

Commits
-------

9b98c63 DX: PhpdocAlignFixer - refactor to use DocBlock
@kubawerlos kubawerlos deleted the dx-refactor-phpdoc-align-to-use-docblock branch January 4, 2019 13:41
keradus added a commit that referenced this pull request Jan 4, 2019
This PR was merged into the 2.12 branch.

Discussion
----------

DX: remove Utils::splitLines

This is the surprise mentioned [here](#4205 (comment)).

Class `Utils` is internal so no [backward compatibility](https://symfony.com/doc/current/contributing/code/bc.html) break.

Commits
-------

4cf6902 DX: remove Utils::splitLines
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants