-
-
Notifications
You must be signed in to change notification settings - Fork 159
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
dev fails because of astro #1064
Comments
Hey @Jessewb786 Thanks for sharing this. We have seen a few issues with this recently (check on Discord for convo) Can you try this https://www.eventcatalog.dev/docs/development/guides/upgrading#having-problems (Delete node_modules, lock files, .eventcatalog-core) and try again? |
@boyney123 Unfortunately, after trying that, I still get the following:
P.S: Everything works fine again if I rollback to 2.16.6. The problem starts specifically with 2.17.0 with the introduction of Astro 5. |
I also encountered this problem and I noticed it fails when project's |
@eps90 Indeed, and what I find myself having to do is make everything is lazily computed so that when astro starts with the eventcatalog.config.js, it doesn't compute everything. And then I still have the issue that I have dynamically resolved generators in my eventcatalog.config.js which are not needed when I run dev, so then I have this frankly sad if check:
This lets me run the generate from package.json and also lets me run dev. I would love to know if there is some other way of integrating dynamically resolved generators. |
I'm not sure how to replicate this and @carlosallexandre looked too. Going to close for now, as the latest version of the catalog does not seem to give this error? If you can find a way we can replicate please feel free to open the issue. |
@boyney123 My comment above was about the newest version of EventCatalog and I provided an information how to replicate that. |
Ah, thanks @eps90 This might help to replicate @carlosallexandre #1064 (comment)? |
@eps90 I tried this way:
|
@carlosallexandre Sorry for no response, I send some detailed steps to reproduce on Monday, if that's ok. I'm AFK until then |
Thank you @eps90 |
This issue is no longer occurring for me on the latest versions. |
I tried this:
Tried to run
"dev": "eventcatalog dev"
on a clean event-catalog after annpm install
.The issue seems to be regarding @astrojs/mdx not being imported.
This happened:
I expected this:
I, of course, expected the service to start-up normally on port 3000.
Is there a workaround?
No response
Anything else?
No response
EventCatalog Version
"@eventcatalog/core": "2.18.5"
Node.js Version
v20.16.0
Platform(s)
MacOS
Community Notes
The text was updated successfully, but these errors were encountered: