Skip to content

feat: Add missing semicolon to use statement of superClass#6350

Merged
bshaffer merged 2 commits intogoogleapis:mainfrom
razvanphp:fix-superclass-use
Jun 4, 2025
Merged

feat: Add missing semicolon to use statement of superClass#6350
bshaffer merged 2 commits intogoogleapis:mainfrom
razvanphp:fix-superclass-use

Conversation

@razvanphp
Copy link
Copy Markdown
Contributor

Without this, generated classes are invalid PHP files.

Screenshot 2025-03-12 at 06 29 34

@razvanphp razvanphp requested a review from a team March 12, 2025 04:32
@razvanphp razvanphp force-pushed the fix-superclass-use branch from b3a11ad to 7d356c9 Compare March 12, 2025 04:41
@razvanphp
Copy link
Copy Markdown
Contributor Author

@bshaffer please also merge this one, it's a no-brainer, this is a bug.

@bshaffer
Copy link
Copy Markdown
Contributor

bshaffer commented Jun 4, 2025

Wow that's strange, as we have a test which verifies all files generated are valid PHP. I don't know how a bug like this could be avoiding getting caught. I'll look into it, but in the meantime thanks for finding this!!

@bshaffer bshaffer enabled auto-merge (squash) June 4, 2025 14:54
@bshaffer bshaffer merged commit da41666 into googleapis:main Jun 4, 2025
11 checks passed
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.

2 participants