Skip to content

Conversation

rtablada
Copy link

@rtablada rtablada commented Oct 9, 2025

Description

This adds a compiler for Ember/Glimmer components as well as examples for use in both a Vite and Webpack build tooling for Ember

Linked Issues

Additional context

@NullVoxPopuli may be able to help get things over the 🏁, this was a WIP from a hackathon/side project that I have not been able to get back to.

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@userquin
Copy link
Member

userquin commented Oct 9, 2025

Run the build command, ember.d.ts types should be added to the subpackage exports at package json file.

Try to reduce both ember examples to minimal required code to test the icons 🙏 (update root .gitignore to exclude ember stuff at examples).

@NullVoxPopuli
Copy link

I can do that -- @rtablada can you add me as a collaborator on your fork? <3

@NullVoxPopuli
Copy link

NullVoxPopuli commented Oct 9, 2025

hm, any way I can see details on why socket.io is saying my deps are bad? 😢

image

@NullVoxPopuli
Copy link

I thiiiink this is ready for review?
Happy to do whatever else is needed though

@userquin
Copy link
Member

userquin commented Oct 9, 2025

Did you run the build command from root? ./types/ember subpackage export should be added to the package.json by tsdown.

@userquin
Copy link
Member

userquin commented Oct 9, 2025

hm, any way I can see details on why socket.io is saying my deps are bad? 😢

I'm getting also forbidden message 🤔

Check https://socket.dev/?utm_medium=gh, you have some hints at the end of the page

@userquin userquin marked this pull request as ready for review October 9, 2025 20:59
Copy link

pkg-pr-new bot commented Oct 9, 2025

Open in StackBlitz

npm i https://pkg.pr.new/unplugin-icons@419

commit: 2a2da4d

@userquin
Copy link
Member

userquin commented Oct 9, 2025

We still need the ember types at subpackage exports, just run pnpm build from root folder and push the changes added by tsdown build.

@NullVoxPopuli
Copy link

how do you mean? i did ? I think?

image

@userquin
Copy link
Member

userquin commented Oct 9, 2025

how do you mean? i did ? I think?

I was writting the comment and you push the commit 👍 now it is fine

@userquin userquin changed the title Support For Ember with Webpack and Vite Example feat(compiler): add support for Ember Oct 9, 2025
@userquin
Copy link
Member

userquin commented Oct 9, 2025

Will do a review tmr from laptop

@userquin
Copy link
Member

userquin commented Oct 9, 2025

can you run pnpm lint from root? Maybe we need to change also eslint config file (rules)

@NullVoxPopuli
Copy link

there is a failing lint at the moment, because one of the lint rules has assumed all .map functions are Array.prototype.map 🙈

@NullVoxPopuli
Copy link

I pushed a little ignore with details about why in my most recent commit

@userquin
Copy link
Member

Looks like ember doing weird things without test/index.html entry point (ember plugin checking some entries in the rollup configuration entry), check the error below (dev server working, only tested with Vite):

dev server error dev server error

Copy link
Member

@userquin userquin left a comment

Choose a reason for hiding this comment

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

once released we should add a link here https://github.com/unplugin/unplugin-icons/blob/main/examples/README.md adding the corresponding StackBlitz script file and command (at least for Vite)

@NullVoxPopuli
Copy link

check the error below (dev server working, only tested with Vite):

yea, I need to fix that 🙈 it's very easy to check if that file exists before adding the entrypoint, I just keep forgetting 🫠

we should add a link here

I can do that! <3

@NullVoxPopuli
Copy link

thanks for the reminder! embroider-build/embroider#2584 <3

@userquin
Copy link
Member

userquin commented Oct 10, 2025

@NullVoxPopuli can you find me at discord at antfu server unplugin-icons channel? I have a few questions about Ember and IDE support...

https://discord.gg/DMn6tWHV

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