-
Notifications
You must be signed in to change notification settings - Fork 414
[PHP] Align @php-wasm/{web,node}-5-2 with workspace and add READMEs #3506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,33 @@ | ||||||
| # @php-wasm/node-5-2 | ||||||
|
|
||||||
| PHP 5.2 WebAssembly binaries for Node.js (legacy). | ||||||
|
|
||||||
| This package contains: | ||||||
|
|
||||||
| - JSPI and Asyncify variants of PHP 5.2 compiled to WebAssembly | ||||||
|
|
||||||
| No bundled extensions (intl, Xdebug, Redis, Memcached) — calling the | ||||||
| corresponding getter functions throws. | ||||||
|
|
||||||
| ## Installation | ||||||
|
|
||||||
| ```bash | ||||||
| npm install @php-wasm/node-5-2 | ||||||
| ``` | ||||||
|
|
||||||
| ## Usage | ||||||
|
|
||||||
|
||||||
| ```typescript | ||||||
| import { getPHPLoaderModule } from '@php-wasm/node-5-2'; | ||||||
|
|
||||||
| const loaderModule = await getPHPLoaderModule(); | ||||||
| ``` | ||||||
|
|
||||||
| ## Related Packages | ||||||
|
|
||||||
| - [@php-wasm/node](https://www.npmjs.com/package/@php-wasm/node) - Main package (requires version packages) | ||||||
|
||||||
| - [@php-wasm/node](https://www.npmjs.com/package/@php-wasm/node) - Main package (requires version packages) | |
| - [@php-wasm/node](https://www.npmjs.com/package/@php-wasm/node) - Main package (requires a versioned build package such as `@php-wasm/node-5-2`) |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,33 @@ | ||||||
| # @php-wasm/web-5-2 | ||||||
|
|
||||||
| PHP 5.2 WebAssembly binaries for the web (legacy). | ||||||
|
|
||||||
| This package contains: | ||||||
|
|
||||||
| - JSPI and Asyncify variants of PHP 5.2 compiled to WebAssembly | ||||||
|
|
||||||
| No bundled extensions (intl, Xdebug, Redis, Memcached) — calling the | ||||||
| corresponding getter functions throws. | ||||||
|
||||||
| corresponding getter functions throws. | |
| corresponding getter functions throw. |
Copilot
AI
Apr 17, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The example uses await at top level, which will fail in many CommonJS / non-top-level-await environments. Consider wrapping in an async function (or explicitly noting that the example requires ESM + top-level await) so users can copy/paste successfully.
Copilot
AI
Apr 17, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The phrase “requires version packages” is ambiguous. Consider clarifying what consumers should install/import (e.g., ‘requires a versioned build package like @php-wasm/web-5-2 / @php-wasm/web-8-5’) to reduce confusion.
| - [@php-wasm/web](https://www.npmjs.com/package/@php-wasm/web) - Main package (requires version packages) | |
| - [@php-wasm/web](https://www.npmjs.com/package/@php-wasm/web) - Main package (requires a versioned build package such as `@php-wasm/web-5-2` or `@php-wasm/web-8-5`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammar: ‘getter functions throws’ should be ‘getter functions throw’.