-
Notifications
You must be signed in to change notification settings - Fork 214
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
feat: upgraded to node v18, added .nvmrc and updated workflows #1084
Conversation
…into bilalqamar95/node-v18-upgrade
…lalqamar95/node-v18-upgrade
…lalqamar95/node-v18-upgrade
…x/frontend-app-learning into bilalqamar95/node-v18-upgrade
…lalqamar95/node-v18-upgrade
Hello, Would it be possible to upgrade frontend-build in this PR to at least 12.8.0 (similar to this openedx/frontend-app-authoring/pull/464 ) hence from 12.7.0 frontend-build is upgraded to use node 18. I just think doing it through this PR would be more efficient and I am happy to help if there are any blockers to this PR. For more context: #1047 (comment) CC: @itsjeyd |
…nto bilalqamar95/node-v18-upgrade
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #1084 +/- ##
==========================================
- Coverage 87.01% 86.96% -0.05%
==========================================
Files 253 253
Lines 4382 4382
Branches 1108 1108
==========================================
- Hits 3813 3811 -2
- Misses 550 552 +2
Partials 19 19 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense to me
Hey, just a note about frontend-build in the comment above, would that befeasible to do this PR?
…On Tue, May 9, 2023, 8:09 PM Jansen Kantor ***@***.***> wrote:
***@***.**** approved this pull request.
Makes sense to me
—
Reply to this email directly, view it on GitHub
<#1084 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD42PH3BEZZBYF6WYF7HOBLXFJ24NANCNFSM6AAAAAAV3OTERU>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
…nto bilalqamar95/node-v18-upgrade
Sure @ghassanmas, I have updated |
…lqamar95/node-v18-upgrade
I temporarily removed |
…lalqamar95/node-v18-upgrade
Tests re-enabled. |
* feat: upgraded to node v18, added .nvmrc and updated workflows * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated jest & fixed failing tests * refactor: updated lmsPact failing test cases * refactor: updated frontend-build version * Merge branch master of github.com:edx/frontend-app-learning into bilalqamar95/node-v18-upgrade --------- Co-authored-by: mashal-m <[email protected]>
* feat: upgraded to node v18, added .nvmrc and updated workflows * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated jest & fixed failing tests * refactor: updated lmsPact failing test cases * refactor: updated frontend-build version * Merge branch master of github.com:edx/frontend-app-learning into bilalqamar95/node-v18-upgrade --------- Co-authored-by: mashal-m <[email protected]>
…dx#1084) * feat: upgraded to node v18, added .nvmrc and updated workflows * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated jest & fixed failing tests * refactor: updated lmsPact failing test cases * refactor: updated frontend-build version * Merge branch master of github.com:edx/frontend-app-learning into bilalqamar95/node-v18-upgrade --------- Co-authored-by: mashal-m <[email protected]>
…dx#1084) * feat: upgraded to node v18, added .nvmrc and updated workflows * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated jest & fixed failing tests * refactor: updated lmsPact failing test cases * refactor: updated frontend-build version * Merge branch master of github.com:edx/frontend-app-learning into bilalqamar95/node-v18-upgrade --------- Co-authored-by: mashal-m <[email protected]>
…dx#1084) * feat: upgraded to node v18, added .nvmrc and updated workflows * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * fix: add workaround in json file * build: remove jestEnv line from json * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated packages * refactor: updated jest & fixed failing tests * refactor: updated lmsPact failing test cases * refactor: updated frontend-build version * Merge branch master of github.com:edx/frontend-app-learning into bilalqamar95/node-v18-upgrade --------- Co-authored-by: mashal-m <[email protected]>
Ticket
Upgrade Node Js from 16 to 18
Description
frontend-build
to v12.8edx-internal PR
https://github.com/edx/edx-internal/pull/8336
This needs to be merged together.