Skip to content

Conversation

@s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Jul 10, 2024

Adds the module folder to the build output when building packages in root-level.

@s1gr1d s1gr1d requested review from Lms24 and mydea July 10, 2024 08:36
Comment on lines +31 to +42
What is working:

- Error Reporting

What is partly working:

- Tracing by setting `tracesSampleRate`

What is not yet(!) included:

- Source Maps
- Connected Traces
Copy link
Member Author

Choose a reason for hiding this comment

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

I will add a more detailed description later. Just wanted to fix the module folder output right now and add some basic information along the way.

"^build:transpile",
"^build:types"
"^build:types",
"build:types"
Copy link
Contributor

Choose a reason for hiding this comment

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

Transpilation usually shouldn't depend on the same packages types. Is there a reason you added this?

Copy link
Member Author

@s1gr1d s1gr1d Jul 10, 2024

Choose a reason for hiding this comment

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

This is because I am building the module with nuxt-module-build and this already relies on the types. I am getting the build warning Potential missing package.json files: build/types/index.types.d.ts if there are no types yet.

@s1gr1d s1gr1d merged commit a79c566 into develop Jul 10, 2024
@s1gr1d s1gr1d deleted the sig/fix-build-nuxt branch July 10, 2024 08:58
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.

4 participants