Skip to content

This library is now a partial-polyfill for RFC#1068#440

Merged
NullVoxPopuli merged 4 commits intomasterfrom
nvp/built-in-built-ins
Dec 23, 2025
Merged

This library is now a partial-polyfill for RFC#1068#440
NullVoxPopuli merged 4 commits intomasterfrom
nvp/built-in-built-ins

Conversation

@NullVoxPopuli
Copy link
Copy Markdown
Collaborator

@NullVoxPopuli NullVoxPopuli commented Sep 2, 2025

When using ember-source 6.8.0-beta.1 or higher, most of the code in this library goes away.

Behavior on earlier ember-source versions is unchanged

Success for this PR will be that:

  • no changes to tests
  • CI for ember-source 6.8.0-beta.1+ passes (this PR started out with throwing on these versions

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This library used to be js + .d.ts, and it was migrated to ts.

I've converted it back, because we now have too much dynamism, and we actually want wholly different implementations to be indistinguishable. from each other, which is something declarations are better at supporting than TS-as-Source

@NullVoxPopuli NullVoxPopuli force-pushed the nvp/built-in-built-ins branch from 93f1e70 to 51cc75a Compare December 1, 2025 17:37
@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review December 2, 2025 15:45
@NullVoxPopuli NullVoxPopuli force-pushed the nvp/built-in-built-ins branch 2 times, most recently from cc1724e to a897457 Compare December 2, 2025 20:03
"@babel/plugin-transform-typescript": "^7.24.4",
"@babel/runtime": "^7.26.10",
"@embroider/addon-dev": "^4.3.1",
"@embroider/macros": "^1.19.5",
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

devDep needed for types

@NullVoxPopuli NullVoxPopuli force-pushed the nvp/built-in-built-ins branch from 36643e0 to c305d15 Compare December 2, 2025 22:21
Copy link
Copy Markdown
Collaborator Author

@NullVoxPopuli NullVoxPopuli Dec 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it isn't possible to build and use those built types from source.

and it's not worth any more of my time to continue trying to coerce TS into something it's not designed to do.

(already spent too much on this already haha)

@NullVoxPopuli NullVoxPopuli force-pushed the nvp/built-in-built-ins branch from c305d15 to 283a423 Compare December 2, 2025 22:23
@johanrd
Copy link
Copy Markdown

johanrd commented Dec 17, 2025

great, this is possibly a prerequisite for making @universal-ember/table, ember-mobile-menu and ember-stargate pure v1-addon-less, as well?

@NullVoxPopuli
Copy link
Copy Markdown
Collaborator Author

Ye!

@NullVoxPopuli NullVoxPopuli added the enhancement New feature or request label Dec 23, 2025
@NullVoxPopuli NullVoxPopuli merged commit d2cfa1a into master Dec 23, 2025
20 of 21 checks passed
@NullVoxPopuli NullVoxPopuli deleted the nvp/built-in-built-ins branch December 23, 2025 17:24
@github-actions github-actions bot mentioned this pull request Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants