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

Update all Released MFEs to use Node 20 #267

Open
1 of 26 tasks
Tracked by #363
feanil opened this issue Feb 8, 2024 · 1 comment
Open
1 of 26 tasks
Tracked by #363

Update all Released MFEs to use Node 20 #267

feanil opened this issue Feb 8, 2024 · 1 comment
Assignees
Labels
epic Large unit of work, consisting of multiple tasks maintenance Routine upkeep necessary for the health of the platform

Comments

@feanil
Copy link
Contributor

feanil commented Feb 8, 2024

Description

See the parent issue for the motivation behind this upgrade: openedx/platform-roadmap#363.

General tasks

Upgrade process

For each MFE, one should do the following, in order:

  1. Create and merge a PR that does nothing else but add Node 20 to the test matrix in a non-blocking manner (using continue-on-error).

    Example PR:

    test: Add Node 20 to CI matrix frontend-template-application#743

  2. If failures are detected on Node 20, create a second PR that addresses those failures while making sure Node 18 still works correctly. Get it merged.

  3. Now create a second (or third) PR that:

    • Updates .nvmrc to use Node 20

    • Regenerates package-lock.json from scratch using it (hint: nvm use && rm package-lock.json && npm install.)

    • Migrates the lockfile check workflow to v3.

    • Removes continue-on-error so Node 20 tests must pass

    • Fixes any issues in the codebase that arise from the above. At this point both NPM versions are supported, but Node 20 is used for development.

    Example PR:

    build: Upgrade to Node 20 frontend-template-application#744

  4. Create a final PR that removes Node 18 from the test matrix. If the MFE is officially supported by Tutor, wait to merge it until tutor-mfe is flipped over to use Node 20 for its builds (see the tracking issue).

    Example PR:

    test: Remove support for Node 18 frontend-template-application#745

The following task list will track the above for individual MFEs:

Tutor-supported MFEs

  1. arbrandes
  2. 0 of 3
    maintenance
  3. 0 of 3
    maintenance
  4. 0 of 3
    maintenance
  5. 0 of 3
    maintenance
  6. 0 of 3
    maintenance
  7. 0 of 3
    maintenance
  8. 0 of 3
    maintenance
  9. 0 of 3
    maintenance
  10. 0 of 3
    maintenance
  11. 0 of 3
    maintenance
  12. 0 of 3
    maintenance
  13. 0 of 3
    maintenance
Copy link

Thanks for your submission, @openedx/open-edx-project-managers will review shortly.

@kdmccormick kdmccormick transferred this issue from openedx/public-engineering Jun 13, 2024
@kdmccormick kdmccormick transferred this issue from openedx/platform-roadmap Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic Large unit of work, consisting of multiple tasks maintenance Routine upkeep necessary for the health of the platform
Projects
Status: In Progress
Status: In progress
Development

No branches or pull requests

2 participants