Skip to content

Conversation

@JakeQZ
Copy link
Contributor

@JakeQZ JakeQZ commented Jan 29, 2026

These mostly just duplicate the PHP documenation. Developers are unlikely to refer to these for the PHP documentation, and they can still inspect the return type to confirm that, e.g., false will not be returned.

Instead, only include the type information.

This reduces the size of the generated code from 6.27Mb to 2.79Mb.

Part of #706.

@JakeQZ JakeQZ mentioned this pull request Jan 29, 2026
@JakeQZ
Copy link
Contributor Author

JakeQZ commented Jan 29, 2026

This will probably fail the CI checks because I have not updated the generated files, and I see it does diff with them.

@JakeQZ JakeQZ marked this pull request as draft January 29, 2026 09:10
@JakeQZ
Copy link
Contributor Author

JakeQZ commented Jan 29, 2026

There's a PHPStan warning which appears to have arisen as a result of a change to Stan. I'll fix that with a separate PR...

@JakeQZ
Copy link
Contributor Author

JakeQZ commented Jan 29, 2026

There's a PHPStan warning which appears to have arisen as a result of a change to Stan. I'll fix that with a separate PR...

#708

@shish shish force-pushed the task/docblock-comments branch from 050daf9 to 3cdc92f Compare January 31, 2026 12:45
These mostly just duplicate the PHP documenation.  Developers are unlikely to
refer to these for the PHP documentation, and they can still inspect the
return type to confirm that, e.g., `false` will not be returned.

Instead, only include the type information.

This reduces the size of the generated code from 6.27Mb to 2.79Mb.

Part of thecodingmachine#706.
@shish shish force-pushed the task/docblock-comments branch from 3cdc92f to 16d0f44 Compare January 31, 2026 12:50
@shish shish marked this pull request as ready for review January 31, 2026 12:54
@shish
Copy link
Collaborator

shish commented Jan 31, 2026

  • updated the code to handle changes in upstream projects, getting the build green again
  • rebased this PR on top of that
  • fixed unit tests
  • I'mma interpret @OskarStark 's rocket emoji as positive feelings 👀 With two maintainers feeling positively I'll go ahead and merge this subjective change

@shish shish merged commit 31553ee into thecodingmachine:master Jan 31, 2026
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