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 GH workflow config to update node 14.x > 16.x #1898

Merged
merged 1 commit into from
May 10, 2022

Conversation

PikachuEXE
Copy link
Collaborator

Pull Request Type
Please select what type of pull request this is:

  • CI Update

Related issue
N/A

Description
Update all GH Action config to use Node 16.x instead of 14.x
16.x is the latest LTS version now

Screenshots (if appropriate)
N/A

Testing (for code that is not small enough to be easily understandable)
No testing done...
Well I am using Node 16.x locally for running FT in dev

Desktop (please complete the following information):

  • OS: N/A
  • OS Version: N/A
  • FreeTube version: N/A

Additional context
Add any other context about the problem here.

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 17, 2021
@peepo5
Copy link
Contributor

peepo5 commented Nov 25, 2021

Weren't there problems with node 16? Or am I just imagining things

@PikachuEXE
Copy link
Collaborator Author

What problems?

@ChunkyProgrammer
Copy link
Member

Weren't there problems with node 16? Or am I just imagining things

Only thing I can think of is package-lock conflicts (we're not using package-lock anymore and 16 is the currently active version so I think this change is reasonable)

@PikachuEXE
Copy link
Collaborator Author

@absidue
Copy link
Member

absidue commented Apr 10, 2022

There are issues on node 17 (we need to use a newer webpack version to fix it AFAIK), 16 is fine though.

Copy link
Member

@PrestonN PrestonN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like we had to revert to v14 for some reason, but I can't remember. I think it was a temp solution anyways. If this causes problems then we'll look at it again I guess.

@PrestonN PrestonN merged commit 24bce11 into FreeTubeApp:development May 10, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label May 10, 2022
@PikachuEXE PikachuEXE deleted the ci/update-node branch May 11, 2022 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants