Skip to content

[comments] Remove attribute comments support as comments require standalone line before any doctrine annotation#7582

Merged
TomasVotruba merged 3 commits into
mainfrom
tv-cleanup-comments
Oct 25, 2025
Merged

[comments] Remove attribute comments support as comments require standalone line before any doctrine annotation#7582
TomasVotruba merged 3 commits into
mainfrom
tv-cleanup-comments

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Oct 25, 2025

By conventions, comments must be before doctrine annotations. All other nodes work this way: php-parser does not allow // commments after the node. This should also speedup docblock parsing.

Reverts #5650

@TomasVotruba TomasVotruba marked this pull request as draft October 25, 2025 20:21
@TomasVotruba TomasVotruba marked this pull request as ready for review October 25, 2025 20:29
@TomasVotruba TomasVotruba changed the title remove attribute comments support as comments require standalone line before any doctrine annotation [comments] Remove attribute comments support as comments require standalone line before any doctrine annotation Oct 25, 2025
@TomasVotruba TomasVotruba merged commit 926706f into main Oct 25, 2025
60 of 61 checks passed
@TomasVotruba TomasVotruba deleted the tv-cleanup-comments branch October 25, 2025 20:37
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant