Skip to content

fix: add type-fest as dependency due to build error#9317

Merged
benelan merged 1 commit intohotfixfrom
benelan/9307-typefest-build-error
May 11, 2024
Merged

fix: add type-fest as dependency due to build error#9317
benelan merged 1 commit intohotfixfrom
benelan/9307-typefest-build-error

Conversation

@benelan
Copy link
Copy Markdown
Contributor

@benelan benelan commented May 11, 2024

Related Issue: #9307

Summary

Add type-fest to the dependencies of @esri/calcite-components to prevent build errors. type-fest can also be used to simplify/remove the preact type generator in the future.

BEGIN_COMMIT_OVERRIDE
fix: add type-fest as dependency due to build error
END_COMMIT_OVERRIDE

@github-actions github-actions Bot added the chore Issues with changes that don't modify src or test files. label May 11, 2024
@benelan benelan added the skip visual snapshots Pull requests that do not need visual regression testing. label May 11, 2024
@jcfranco
Copy link
Copy Markdown
Member

Should this use the fix commit type since it’s addressing a reported bug?

@benelan benelan changed the title build(deps): add type-fest to prevent build errors fix: add type-fest as dependency due to build error May 11, 2024
@benelan benelan merged commit 3819688 into hotfix May 11, 2024
@benelan benelan deleted the benelan/9307-typefest-build-error branch May 11, 2024 03:14
benelan added a commit that referenced this pull request May 14, 2024
**Related Issue:** #9307

## Summary

Add `type-fest` to the dependencies of `@esri/calcite-components` to
prevent build errors. `type-fest` can also be used to simplify/remove
the [preact type
generator](https://github.com/Esri/calcite-design-system/blob/27a82dcaf8b091c609b3bf765d8c8617c447d8e7/packages/calcite-components/support/preact.ts#L8)
in the future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Issues with changes that don't modify src or test files. skip visual snapshots Pull requests that do not need visual regression testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants