-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
Error when evaluating SSR module "astro-icon" #231
Comments
I just started hitting this as well. The Astro build command is working fine, but I have a GitHub Action that runs vitests that started failing this afternoon with this error. |
It has something to do with the latest release of Cheerio |
Facing this as well 👍🏽 |
Not sure how safe it is, but this worked for me (I'm just learning astro at this moment): |
Came to comment this. Works as a temporary solution, will use this until they get fixed. |
Another way to solve it temporarily is to use resolutions in package.json (I struggled to get the overrides thing to work in a monorepo):
|
This problem is already solved in @iconify/tools version 4. "overrides": {
"astro-icon": {
"@iconify/tools": "^4.0.0"
}
} |
It worked for me by installing the latest version, npm install @iconify/tools --save |
I still get an error without:
|
|
I'm having the same issue. |
Same here. It did not work by any other way... |
@rderimay @adnansmajlovic If you added the @iconify/tools override, don't forget to run |
yes sure. Did it. Event deleted the |
On pnpm I had to do this instead "pnpm": {
"overrides": {
"@iconify/tools": "^4.0.5"
}
} |
Thanks! 🙏🏻 This is why it did not work as I am using pnpm. (ping @Stretsh) |
still not fixed, astro still uses version 3.0.7 and even if you change the version to 4.0.5 I still get the error with cheerio... |
For me it worked fine after using that override in my package.json and then running |
Thanks The issue is solved now |
"overrides": { Thanks The issue is solved now |
This worked for me. Amazing thanks! |
wow,thank you ,use code fix the bug |
Facing same error and |
Seem to be fixed in astro-icon@1.1.1 by #230 bumping to @iconify/tools@^4.0.4 At least i don't have build issues anymore on my side |
I believe this is resolved. Closing. |
What version of
astro-icon
are you using?v1.1.0
Astro Info
Note
I had to take out
astro-icon
Integration or it won't even run the commandIf this issue only occurs in one browser, which browser is a problem?
Every
Describe the Bug
What's the expected result?
It should work
Link to Minimal Reproducible Example
I was not able to recreate it on StackBlitz, but doing
It gives me the same error
The text was updated successfully, but these errors were encountered: