Skip to content
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

Chore: Upgrade Angular #211

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

sorenjohanson
Copy link
Contributor

closes #192

@JannikStreek
Copy link
Member

JannikStreek commented Nov 20, 2024

Unknown error: Error [ERR_REQUIRE_ESM]: require() of ES Module /home/node/app/node_modules/wrap-ansi/index.js from /home/node/app/node_modules/cliui/build/index.cjs not supported.
Instead change the require of index.js in /home/node/app/node_modules/cliui/build/index.cjs to a dynamic import() which is available in all CommonJS modules.
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

When I run yarn start. or yarn test Don't you have that issue?

@JannikStreek
Copy link
Member

@sorenjohanson can you fix the conflicts and fix the build errors so we can merge this?

@sorenjohanson
Copy link
Contributor Author

Done, though idk if the tests will fail again as they seem to be getting the cli error (even though I can't reproduce it locally anymore)

@JannikStreek
Copy link
Member

what's the status here?

@sorenjohanson
Copy link
Contributor Author

Can you still get the cli error to reproduce when checking out? I can't, so I have no idea why yarn lint and test are failing.

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.

chore: upgrade angular
2 participants