Open chriskrycho opened 1 year ago
FYI, the ember-cli-sri
repo is archived.
@chriskrycho Seems like a part is missing from the ember-fetch
entry in the list above?
Oh, good catches on both points. Updating!
@chriskrycho @bertdeblock I'd like to help out with ember-welcome-page
.
I think we need to first update the repo (CI, dependencies, and minimum support for Node & Ember), before we introduce TypeScript, though. This would help with future maintenance.
Is it okay if I temporarily own the project and make a few pull requests to keep the project up-to-date and easy to maintain?
Please do! I'll mark is at "taken" here and am happy to help with any TS work there.
@chriskrycho Thanks!
@emberjs/typescript-core Can this be closed?
I think we've completed the high-priority items on this list but not everything. Checking in with @wagenet
RFC 0800 Tracking Issue
Implement RFC 0800: TypeScript Adoption Plan.
Enable TS blueprints
Publish types
We do not need to publish types for every package in Ember's dependencies—only for the packages which are part of its programmatic public APIs. An example of a dependency we don’t need to ship types with:
ember-cli-dependency-checker
is integrated into the classic Ember CLI build pipeline, but it doesn’t expose any APIs for users to call. The following list is comprised of the items in the default blueprints filtered for that criteria.These items will be checked off if they have been "claimed"—please comment if you'd like to claim one! We will track their status via links to the issues/PRs which implement them.
Status: Last updated 2022/12/13, with the dependencies from the app and v1 addon blueprints.
ember-source
(i.e. Ember.js)ember-data
, including:ember-data
@ember-data/adapter
@ember-data/canary-features
@ember-data/debug
@ember-data/deprecations
@ember-data/model
@ember-data/record-data
@ember-data/serializer
@ember-data/store
@ember/optional-features
: https://github.com/emberjs/ember-optional-features/issues/329@ember/test-helpers
: https://github.com/emberjs/ember-test-helpers/issues/1268@glimmer/component
– what we ship here is actually somewhat wrong, but not in a blocking way@glimmer/tracking
– what we ship here is actually right from a TS POV (unlike the runtime POV!)ember-auto-import
– already ships types :tada:ember-cli
: ember-cli/ember-cli#9996ember-cli-app-version
ember-cli-babel
ember-cli-htmlbars
: https://github.com/ember-cli/ember-cli-htmlbars/issues/753ember-cli-inject-live-reload
ember-cli-terser
ember-fetch
– Note: this one is complicated. It already ships types, but badly. Please extra reach out if you pick this one up: a path forward exists but isn't necessarily obvious!ember-load-initializers
– already ships types! :tada:ember-page-title
: https://github.com/ember-cli/ember-page-title/issues/239ember-qunit
: https://github.com/emberjs/ember-qunit/issues/957ember-resolver
: https://github.com/ember-cli/ember-resolver/issues/795ember-template-lint
ember-welcome-page
: ember-cli/ember-welcome-page#387loader.js
[^loader]qunit
– has reasonably-good types (which we help maintain) on DefinitelyTyped and is not a package we controlqunit-dom
– already ships types! :tada:[^loader]: Debatably, anyway: we could work on dropping this instead, and its public API usage is intended to be minimal; but right now it is a public API.