-
Notifications
You must be signed in to change notification settings - Fork 1
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
Incrementally upgrade edx-platform NPM dependencies #105
Comments
This is listed as Blocked on the FED-BOM board; @ishahroz , what is it currently waiting on? |
we are trying to upgrade webpack from v2 to v4. Facing issues in underscore configs with this upgrade. |
Many major package updates were done as a part of the Node 18 upgrade and the package.json now compiles properly without issues thanks to some cleanup in the proctoring dependencies which were previously using See openedx/edx-platform#34873 and https://github.com/openedx/edx-platform/pulls?q=is%3Apr+author%3Aapp%2Frenovate+is%3Aclosed for some recent evidence of this. |
Many of the NPM packages used in edx-platform are very stale, and attempts to upgrade many of them at once have generally failed. Pick a tool(s) and process of incrementally upgrading subsets of the dependencies a little at a time, to see if we can gradually make the situation better. The general process should look something like:
One tool that attempts to make such a project easier is npm-check-updates:
npm-check is another option, there may be others as well. It's probably worth spending a few hours researching and comparing the options here.
Once a tool and process are finalized, start using it in edx-platform to update the package*.json files. Don't try to change too many things in one deployment, we don't want to make it too hard to track down what may have caused a bug that slips past tests to affect stage or prod.
If all goes reasonably well, document the process so we can repeat it in other repositories which are falling a little behind in their NPM package updates.
Related tasks
The text was updated successfully, but these errors were encountered: