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

[7.3] MethodArgumentSpaceFixer - fix comma after heredoc-end #4342

Merged

Conversation

gharlan
Copy link
Contributor

@gharlan gharlan commented Mar 18, 2019

Input:

foo(
    <<<'EOD'
        bar
        EOD
    ,
    'baz'
);

With PHP 7.3 and new config option after_heredoc it is fixed to:

foo(
    <<<'EOD'
        bar
        EOD,
    'baz'
);

(similar to #4191)

@gharlan gharlan force-pushed the method-argument-space-heredoc branch from 0d04a0e to e3c63ae Compare March 18, 2019 23:30
@SpacePossum SpacePossum added this to the 2.15.0 milestone Mar 19, 2019
@SpacePossum
Copy link
Contributor

Thank you @gharlan.

@SpacePossum SpacePossum merged commit e3c63ae into PHP-CS-Fixer:master Mar 19, 2019
SpacePossum added a commit that referenced this pull request Mar 19, 2019
…c-end (gharlan)

This PR was merged into the 2.15-dev branch.

Discussion
----------

[7.3] MethodArgumentSpaceFixer - fix comma after heredoc-end

Input:

```php
foo(
    <<<'EOD'
        bar
        EOD
    ,
    'baz'
);
```

With PHP 7.3 and new config option `after_heredoc` it is fixed to:

```php
foo(
    <<<'EOD'
        bar
        EOD,
    'baz'
);
```

(similar to #4191)

Commits
-------

e3c63ae [7.3] MethodArgumentSpaceFixer - fix comma after heredoc-end
@gharlan gharlan deleted the method-argument-space-heredoc branch March 19, 2019 07:35
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

2 participants