Skip to content

Conversation

zeripath
Copy link
Contributor

@zeripath zeripath commented Aug 1, 2021

The recent change in node 16 has caused webpack-cli to break and thence our CI. Ostensibly this has happened because node 16 is not a LTS release.

This PR changes to use Node 14 which is the LTS for node instead of 16.

Closes #16591

Signed-off-by: Andrew Thornton art27@cantab.net

The recent change in node 16 has caused jest to break and thence our CI.
This PR changes to use Node 14 instead of 16.

Closes go-gitea#16591

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added type/testing topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile labels Aug 1, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #16592 (cfbdf03) into main (72738f0) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16592      +/-   ##
==========================================
- Coverage   45.35%   45.35%   -0.01%     
==========================================
  Files         750      750              
  Lines       85096    85096              
==========================================
- Hits        38599    38597       -2     
- Misses      40252    40254       +2     
  Partials     6245     6245              
Impacted Files Coverage Δ
modules/process/manager.go 72.83% <0.00%> (-2.47%) ⬇️
modules/queue/workerpool.go 53.05% <0.00%> (-1.53%) ⬇️
services/pull/pull.go 41.73% <0.00%> (ø)
modules/git/blame.go 67.14% <0.00%> (+1.42%) ⬆️
modules/util/timer.go 85.71% <0.00%> (+42.85%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b73e421...cfbdf03. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 1, 2021
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
@zeripath zeripath deleted the node-14-instead-of-16 branch December 29, 2022 19:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile type/testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants