Skip to content

Remove duplicate Methods header in exception skeleton#276

Merged
Girgias merged 1 commit intophp:masterfrom
lacatoire:fix-skeleton-duplicate-methods
Feb 15, 2026
Merged

Remove duplicate Methods header in exception skeleton#276
Girgias merged 1 commit intophp:masterfrom
lacatoire:fix-skeleton-duplicate-methods

Conversation

@lacatoire
Copy link
Contributor

Summary

  • Remove duplicated <classsynopsisinfo role="comment">&Methods;</classsynopsisinfo> in skeletons/exceptionname.xml
  • classname.xml has this header once before the constructor include; exceptionname.xml had it twice (lines 33 and 35)

File changed

  • skeletons/exceptionname.xml — 1 line removed

@lacatoire lacatoire requested a review from alfsb as a code owner February 10, 2026 09:16
The classsynopsisinfo Methods header appeared twice in
exceptionname.xml, unlike classname.xml which has it once.
@lacatoire lacatoire force-pushed the fix-skeleton-duplicate-methods branch from 43287e7 to f98c7e2 Compare February 10, 2026 09:19
Copy link
Member

@alfsb alfsb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not use these skeletons, but these changes appear to be simple enough.

@Girgias Girgias merged commit ab95b71 into php:master Feb 15, 2026
10 of 12 checks passed
@lacatoire lacatoire deleted the fix-skeleton-duplicate-methods branch February 15, 2026 20:46
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