Skip to content

Add extensions to imports, fix some example issues#371

Merged
vladislavarsenev merged 7 commits into
trivago:v6from
RobbieTheWagner:fix-examples
Jul 23, 2025
Merged

Add extensions to imports, fix some example issues#371
vladislavarsenev merged 7 commits into
trivago:v6from
RobbieTheWagner:fix-examples

Conversation

@RobbieTheWagner
Copy link
Copy Markdown
Contributor

No description provided.

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

@vladislavarsenev it seems like something is wrong still. The prettier.config.js for the repo itself does not seem to work, and all of the examples have issues like examples/example.tsx: TypeError: traverse is not a function. Any ideas what to do next?

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

@vladislavarsenev @byara I think I got all this sorted out now. We basically needed to make sure we had absolutely no CJS or require in the code. I think I have correctly setup dynamic imports for everything, but let me know if anything seems off.

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

@vladislavarsenev @byara please let me know if you would like any changes or if this looks good. I am eager to have all this resolved so I can work on gjs/gts support 😃

Copy link
Copy Markdown
Collaborator

@byara byara left a comment

Choose a reason for hiding this comment

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

This looks good to me. But if you don't mind I'll wait for @vladislavarsenev to have a final check.

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

Sure thing @byara! @vladislavarsenev please let me know if there are any additional changes you would like.

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

@vladislavarsenev just wanted to bump this. Let me know if you would like any changes made please!

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

@byara apologies for the multiple pings, but do you know if @vladislavarsenev is out? I am quite keen to get this merged so I can work on the gjs/gts support.

@byara
Copy link
Copy Markdown
Collaborator

byara commented Jul 21, 2025

@byara apologies for the multiple pings, but do you know if @vladislavarsenev is out? I am quite keen to get this merged so I can work on the gjs/gts support.

Sorry @RobbieTheWagner, @vladislavarsenev was on vacation and today he is back. I'm sure he can take care of it 👍

@vladislavarsenev
Copy link
Copy Markdown
Collaborator

Hi @RobbieTheWagner! Sorry, was unable to take a look PR for a while. I'll take a look tomorrow or so. Sounds good?

@RobbieTheWagner
Copy link
Copy Markdown
Contributor Author

Yes, thanks @vladislavarsenev! I appreciate you taking the time 😃

@vladislavarsenev vladislavarsenev self-requested a review July 22, 2025 18:30
Copy link
Copy Markdown
Collaborator

@vladislavarsenev vladislavarsenev left a comment

Choose a reason for hiding this comment

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

LGTM🚀 I left just a few minor comments. Thank you for your effort!

Comment thread tests/ImportsSeparatedByUser/__snapshots__/ppsi.spec.mjs.snap
Comment thread prettier.config.mjs Outdated
Comment thread src/preprocessors/svelte-preprocessor.ts
Comment thread src/preprocessors/vue-preprocessor.ts
@vladislavarsenev vladislavarsenev merged commit 1905324 into trivago:v6 Jul 23, 2025
3 checks passed
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.

3 participants