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(crwa): Convert Create Redwood App to ESM #9659

Conversation

virtuoushub
Copy link
Collaborator

@virtuoushub virtuoushub commented Dec 12, 2023

@virtuoushub virtuoushub force-pushed the virtuoushub/esm/crwa/covert-to-esm branch from fe4382b to 2732f85 Compare December 12, 2023 02:40
@Tobbe
Copy link
Member

Tobbe commented Dec 12, 2023

@virtuoushub We're running into NX cache issues from time to time on some PRs. That's what you saw here with all the tests failing. I cleared that up and re-ran the tests. Seems to be only one thing that's failing due to ESM now

image

@Tobbe Tobbe added the release:feature This PR introduces a new feature label Dec 12, 2023
@Tobbe Tobbe added this to the v7.0.0 milestone Dec 12, 2023
@virtuoushub
Copy link
Collaborator Author

virtuoushub commented Dec 12, 2023

@virtuoushub We're running into NX cache issues from time to time on some PRs. That's what you saw here with all the tests failing. I cleared that up and re-ran the tests. Seems to be only one thing that's failing due to ESM now

image

awesome! thanks for your help with ea1d5a5

@virtuoushub virtuoushub mentioned this pull request Dec 13, 2023
1 task
@dthyresson dthyresson changed the title chore(crwa): covert to esm chore(crwa): Convert Create Redwood App to ESM Dec 13, 2023
@Tobbe
Copy link
Member

Tobbe commented Dec 13, 2023

This looks good, but please hold off merging until we've got a better plan in place for ESM support.
I'll carry this forward, or hand off as needed.

@Tobbe
Copy link
Member

Tobbe commented Jan 1, 2024

Superseded by #9786

@Tobbe Tobbe closed this Jan 1, 2024
@virtuoushub virtuoushub deleted the virtuoushub/esm/crwa/covert-to-esm branch January 1, 2024 17:48
@jtoar jtoar removed this from the SSR milestone Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:feature This PR introduces a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants