Skip to content

Conversation

Janpot
Copy link
Member

@Janpot Janpot commented Mar 11, 2025

Make sure this works with the new type generation

@Janpot Janpot added the scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label Mar 11, 2025
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 11, 2025
@mui-bot
Copy link

mui-bot commented Mar 11, 2025

Netlify deploy preview

https://deploy-preview-45547--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 67de3ef

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 12, 2025
@Janpot Janpot marked this pull request as ready for review March 12, 2025 09:13
@Janpot Janpot requested a review from a team March 12, 2025 09:13
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Apr 2, 2025
const watcher = fs.watch(sourceDirectory, { recursive: true });
for await (const event of watcher) {
if (event.filename?.endsWith('.d.ts')) {
handleChange(event.filename);
Copy link

Choose a reason for hiding this comment

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

consider normalizing path.resolve(sourceDirectory, event.filename), windows uses \, unix /, path.resolve() ensures you're using the correct format for the current OS

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

Labels

PR: out-of-date The pull request has merge conflicts and can't be merged. scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants