-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Revert "chore: upgrade to node 18" #34496
Conversation
619b0a1
to
2d7e04d
Compare
2d7e04d
to
98b6d6f
Compare
We were seeing issues with XBlock rendering with these changes.
Something about the new webpack config is not loading |
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
This reverts commit eb26333.
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production. |
2U Release Notice: This PR has been deployed to the edX production environment. |
1 similar comment
2U Release Notice: This PR has been deployed to the edX production environment. |
This reverts commit ab2689a, as the upstream Node upgrade was reverted: openedx/edx-platform#34496
This reverts commit eb26333.
This reverts commit eb26333.
revert: revert: "chore: upgrade to node 18" (#34496)
* Revert "chore: upgrade to node 18" * Revert "fix: bump npm from 10.2.x to 10.5.x" This reverts commit b6662d4. * Revert "fix: replace outdated webpack plugins" This reverts commit b204f1b. * Revert "build: Remove the `npm bin` calls from package.json" This reverts commit 4e3dd16. * fix: fix bad conflict resolution. --------- Co-authored-by: Diana Huang <[email protected]>
Reverts #34311
Failing paver assets build:
We're still investigating so might not need this but getting it ready just in-case.