Skip to content

No closing tag#539

Closed
MathiasReker wants to merge 1 commit intophp:masterfrom
MathiasReker:0-no-closing-tag
Closed

No closing tag#539
MathiasReker wants to merge 1 commit intophp:masterfrom
MathiasReker:0-no-closing-tag

Conversation

@MathiasReker
Copy link
Contributor

The closing ?> tag MUST be omitted from files containing only PHP.

The closing ?> tag MUST be omitted from files containing only PHP.
@kamil-tekiela
Copy link
Member

Makes sense, but I would keep a newline at the end of the file.

@localheinz
Copy link
Contributor

For reference, see https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v3.8.0/doc/rules/php_tag/no_closing_tag.rst.

@kamil-tekiela
Copy link
Member

Superseded by #603

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants