chore: target deck.gl and luma.gl 9.4 alphas - #701
Conversation
CoverageStmts: 51.02% | Branch: 42.11% | Funcs: 49.21% | Lines: 51.13%
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 78363682c7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "@loaders.gl/core": "npm:^4.4.3" | ||
| "@loaders.gl/images": "npm:^4.4.3" |
There was a problem hiding this comment.
Raise loaders.gl pins with the deck.gl alpha
On fresh root and website installs, the new deck.gl packages request loaders.gl 4.4.3 or newer, but both package.json resolution blocks still force every loaders.gl package to 4.4.1. Yarn consequently reports YN0060 compatibility mismatches and installs versions outside deck.gl's declared ranges, affecting core, layers, geo-layers, and their loader paths. Update the root and website loaders.gl resolutions and direct ranges alongside this upgrade.
Useful? React with 👍 / 👎.
Summary
9.4.0-alpha.0~9.4.0-alpha.2and luma.gl~9.4.0-alpha.1~9.4.0-alpha.0and the root deck-to-luma peer override to the luma 9.4 alpha baselinemjolnir.jswith deck.gl 9.4 and use Turf 7 for the website/editable-layers dependency graphCompatibility changes
DeckProps['widgets']genericmjolnir.jsat3.1.0-beta.3so deck and community widget event-manager types share one class declarationValidation
yarnyarn lintyarn buildyarn test— 1,414 passed, 9 skippedyarn test-headless— 429 passed, 5 skippedcd website && yarncd website && yarn buildRelease impact
This changes the development baseline on
masteronly. It does not publish a community 9.4 alpha.