Skip to content

Conversation

@adamziel
Copy link
Collaborator

@adamziel adamziel commented Apr 23, 2024

Related to #1287

TypeScript 5.4.5 (well, technically 5.2) gives us the Disposable interface for cleaning up resources.

This PR also includes the changes from CI: Fix intermittent docs build failures:

Building the documentation page often fails. This PR is an exploration
to fix it. It:

  • Patches the typedoc library in node_modules as otherwise it won't
    work with TypeScript 5.4 🙈
  • Removes non-essential TypeDoc plugins (link API reference to MDN,
    cross-link Typedoc modules)
  • Copies the rollup.d.ts files to dist instead of moving them.
    TypeDoc, apparently, expects to find them in their original location:
error TS6053: File '/home/runner/work/wordpress-playground/wordpress-playground/packages/php-wasm/web/src/rollup.d.ts' not found.
  The file is in the program because:
    Root file specified for compilation

@adamziel adamziel added the [Type] Enhancement New feature or request label Apr 23, 2024
@adamziel adamziel requested a review from a team as a code owner April 23, 2024 09:33
Building the documentation page often fails. This PR is an exploration
to fix it. It:

* Patches the typedoc library in `node_modules` as otherwise it won't
work with TypeScript 5.4 🙈
* Removes non-essential TypeDoc plugins (link API reference to MDN,
cross-link Typedoc modules)
* Copies the `rollup.d.ts` files to `dist` instead of moving them.
TypeDoc, apparently, expects to find them in their original location:

```
error TS6053: File '/home/runner/work/wordpress-playground/wordpress-playground/packages/php-wasm/web/src/rollup.d.ts' not found.
  The file is in the program because:
    Root file specified for compilation
```

## Testing Instructions

Restart the CI 5 times, confirm the build worked every single time.
@adamziel adamziel merged commit b9352b2 into trunk Apr 23, 2024
@adamziel adamziel deleted the update-typescript branch April 23, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants