Skip to content

Commit 1ce4532

Browse files
committed
Simpler
1 parent fc04d2c commit 1ce4532

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

text/1068-tracked-collections.md

+3-9
Original file line numberDiff line numberDiff line change
@@ -251,17 +251,11 @@ For some utilities, we can place them under sub-path-exports, such as `@ember/re
251251

252252
### Consumption
253253

254-
When a project wants to use `@ember/reactive`, they would then only need to install the package separately / add it to their `package.json`.
254+
`@ember/reactive` would be another virtual package built in to ember-source, much like `@ember/service`, etc.
255255

256-
The proposed list of compatibility here is only meant as an example -- if implementation proves that more can be supported easier, with less work, that should be pursued, and this part is kind of implementation detail.
256+
For polyfilling, a library could be created that uses `ember-addon.renamed-modules` to instruct our build tooling about a virtual package -- the same technique that `ember-source` uses.
257257

258-
But for demonstration:
259-
- apps pre [version available], would add `@ember/reactive` to their `devDependencies` or `dependencies`
260-
- importing `@ember/reactive` would be handled by ember-auto-import/embroider (as is the case with all v2 addons)
261-
- v1 addons would not be supported
262-
- v2 addons, for maximum compatibility, would need to add `@ember/reactive` to their `dependencies`
263-
- in consuming apps post [version available], this would be optimized away if the version declared in dependencies satisfies the range provided by the consuming app (an optimization that packagers already do, and nothing we need to worry about)
264-
- apps post [version available], would not need to add `@ember/reactive` to their `devDependencies` or `dependencies`, as we can rely on the `ember-addon#renamed-modules` config in ember-source's `package.json`.
258+
Once a project updates to a sufficiently new enough ember-source, the polyfilling library should be deleted.
265259

266260
## How we teach this
267261

0 commit comments

Comments
 (0)