github-actions[bot]
commented
8 months ago Oops, looks like you forgot to add a changeset.
⚠️ Please run rush change and commit the changeset file.
This command will prompt you for a change description and generate a changeset file. You can read more about changesets here.
Remember that you should use the version bump that is appropriate for the change you are making:
| Version bump | Meaning |
|---|---|
patch |
Bug fixes, documentation changes, etc. |
minor |
New features, non-breaking changes |
major |
Breaking changes |
If you are unsure about which version bump to use, please ask in the comments and we will help you out.
javiertoledo
Description
Cleanup of exported methods of the core package to avoid extra levels of indirection. The core index interface has not changed, but I've moved the implementation of the static methods from the
booster.tsfile to theindex.tsto avoid a level of indirection. TheBoosterApptype was made redundant as it was just describing theBoosterclass signature.Changes
index.tsBoosterobject directly without proxying it through theBoosterApptypeBoosterApptype, as theBoosterobject already exports the correct signature. This type was used only to type-check loading user apps from the CLI, and while the type was exported, it was not used in practice. Users use theBoostersingleton exported by the core package instead.Boosterclass, as they were never exported; thus, removing them is not a breaking change.