Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(TypeScript): remove original ts/tsx files and only leave type definition files #1292

Merged
merged 10 commits into from Feb 17, 2022

Conversation

tujoworker
Copy link
Member

@tujoworker tujoworker commented Feb 16, 2022

There are setups and cases where it is not recommended to have .tsx files inside node_modules. Until now we have just published the raw ts source files in order to let projects use the types of our ts components, while we had definition files for the rest.

With this PR – we change this, so we only end up with definitions files.

I will squash all commits on pr merge. But the intention is to get out a beta version first.

Here is the beta release and the ci logs

@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 16, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b08cb95:

Sandbox Source
eufemia-starter Configuration

@gatsby-cloud
Copy link

gatsby-cloud bot commented Feb 16, 2022

Gatsby Cloud Build Report

DNB Eufemia Portal

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 10m

Performance

Lighthouse report

Metric Score
Performance 🔶 64
Accessibility 💚 100
Best Practices 💚 100
SEO 💚 92

🔗 View full report

@tujoworker tujoworker marked this pull request as ready for review February 16, 2022 08:14
Copy link
Contributor

@dinarosv dinarosv left a comment

Choose a reason for hiding this comment

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

So great that you fixed this! 👏

@tujoworker
Copy link
Member Author

A second beta was released in corporation with the issue reporter.

@tujoworker tujoworker merged commit 79bdfb4 into main Feb 17, 2022
@tujoworker tujoworker deleted the feat/build-ts-definitions-only branch February 17, 2022 09:33
tujoworker pushed a commit that referenced this pull request Feb 21, 2022
# [9.23.0-beta.4](v9.23.0-beta.3...v9.23.0-beta.4) (2022-02-21)

### Bug Fixes

* **Dropdown:** disabled tertiary dropdown ([c1b4c4b](c1b4c4b))
* **Styles:** ensure independent styling packages have declared their dependencies ([8b3c230](8b3c230))

### Features

* **esm:** remove esm directory in package build ([f189b62](f189b62))
* **Icon:** add examples for color property ([#1275](#1275)) ([67f8121](67f8121))
* **Tabs:** Enhance parent width handling and remove auto edge detection ([d002f2c](d002f2c))
* **TypeScript:** remove original ts/tsx files and only leave type definition files ([#1292](#1292)) ([79bdfb4](79bdfb4))
@tujoworker
Copy link
Member Author

🎉 This PR is included in version 9.23.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

tujoworker pushed a commit that referenced this pull request Feb 22, 2022
# [9.23.0](v9.22.0...v9.23.0) (2022-02-22)

### Bug Fixes

* **Dropdown:** disabled tertiary dropdown ([c1b4c4b](c1b4c4b))
* **Modal:** add possibility for camelcase props ([4127452](4127452))
* **ProgressIndicator:** css change to fix aliasing artifacts for ProgressIndicator ([ab8859e](ab8859e))
* set Eufemia.version during release ([e74d441](e74d441))
* **Styles:** ensure independent styling packages have declared their dependencies ([8b3c230](8b3c230))
* **Tabs:** align to new UX designs ([794bf70](794bf70))
* **Tag:** fix Tag.Group spacing ([d7b77a2](d7b77a2))

### Features

* **esm:** remove esm directory in package build ([f189b62](f189b62))
* **Icon:** add examples for color property ([#1275](#1275)) ([67f8121](67f8121))
* Separate Drawer and Dialog from Modal ([ebe8e6a](ebe8e6a))
* **Tabs:** Enhance parent width handling and remove auto edge detection ([99ed393](99ed393))
* **TypeScript:** remove original ts/tsx files and only leave type definition files ([#1292](#1292)) ([79bdfb4](79bdfb4))
@tujoworker
Copy link
Member Author

🎉 This PR is included in version 9.23.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants