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

Get build working and run compile in the pipeline #612

Merged
merged 10 commits into from
Nov 23, 2023

Conversation

ChristianHoffmannS2
Copy link
Collaborator

@ChristianHoffmannS2 ChristianHoffmannS2 commented Nov 17, 2023

This should make it possible to import our components in svelte/react using the path "@boiler/ui-library" inside of tsx/ts/svelte files

The compile step is also needed here to get a main.d.ts file so that svelte/react can use this in a typesafe way.

The compile will also be running in pipelines to give us ts errors for ALL ts files and not just the ones included by the build it self
(namely: tests & stories)

Note: the build only includes whats exported in the ui-library/index.ts file.
We should get that file generated instead of handwritten at some point

Copy link
Contributor

@kaikdi kaikdi left a comment

Choose a reason for hiding this comment

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

can you have a look at these font files?
I get that we moved them, but should they be also removed from where they where ?

packages/ui-library/src/index.ts Outdated Show resolved Hide resolved
packages/ui-library/src/index.ts Outdated Show resolved Hide resolved
packages/ui-library/webpack.common.js Outdated Show resolved Hide resolved
@ChristianHoffmannS2 ChristianHoffmannS2 changed the title Get build working Get build working and run compile in the pipeline Nov 23, 2023
@ChristianHoffmannS2 ChristianHoffmannS2 merged commit d1c353d into develop Nov 23, 2023
6 checks passed
@ChristianHoffmannS2 ChristianHoffmannS2 deleted the get-build-working branch November 23, 2023 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants