Skip to content

Commit

Permalink
Fixes #37134 - upgrade to node 18 and npm 8
Browse files Browse the repository at this point in the history
  • Loading branch information
MariaAga committed Feb 5, 2024
1 parent e87de8d commit 39648ff
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/matrix.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"postgresql": ["12"],
"ruby": ["2.7", "3.0"],
"node": ["14"]
"node": ["18"]
}
4 changes: 2 additions & 2 deletions .github/workflows/plugins_react_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,10 @@ jobs:
repository: ${{ matrix.plugin }}
path: ${{ github.workspace }}/projects/plugin
- name: Generate ${{ matrix.plugin }} npm dependencies package-lock
run: npm install --package-lock-only --no-audit
run: npm install --package-lock-only --no-audit --legacy-peer-deps
working-directory: ${{ github.workspace }}/projects/plugin
- name: Install ${{ matrix.plugin }} npm dependencies
run: npm ci --no-audit
run: npm ci --no-audit --legacy-peer-deps
working-directory: ${{ github.workspace }}/projects/plugin
- name: Run ${{ matrix.plugin }} tests
run: npm test
Expand Down
1 change: 1 addition & 0 deletions .npmrc
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
engine-strict=true
legacy-peer-deps=true
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Base container that is used for both building and running the app
FROM quay.io/centos/centos:stream8 as base
ARG RUBY_VERSION="2.7"
ARG NODEJS_VERSION="14"
ARG NODEJS_VERSION="18"
ENV FOREMAN_FQDN=foreman.example.com
ENV FOREMAN_DOMAIN=example.com

Expand Down
7 changes: 2 additions & 5 deletions developer_docs/foreman_dev_setup.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,9 @@

[[prerequisites]]
== Prerequisites
Foreman will run with the following requirements (aside from rubygem dependencies):
Please refer to the `.github/matrix.json` file in the project repository to find the latest supported versions for Ruby, NodeJS, and PostgreSQL. This file is regularly updated to reflect our current support.

* Ruby 2.7.x
* NodeJS 14
* NPM 6.x
* PostgreSQL 13
If you're using NPM version 7.x or higher, you will need to use the `--legacy-peer-deps` flag when installing npm packages. This is due to changes in how NPM handles peer dependencies from version 7 onwards.

It’s recommended to install https://github.com/rbenv/rbenv[rbenv] or https://github.com/rvm/rvm[rvm] to manage ruby, and https://github.com/nvm-sh/nvm[nvm] to manage node and npm on your system.
* For Fedora users, you can use this https://developer.fedoraproject.org/start/sw/web-app/rails.html[guide] for installing: rbenv, Ruby & Bundler.
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"description": "Foreman isn't really a node module, these are just dependencies needed to build the webpack bundle. 'dependencies' are the asset libraries in use and 'devDependencies' are used for the build process.",
"private": true,
"engines": {
"node": ">14.0.0 <16.0.0"
"node": ">=18.0.0 <21.0.0"
},
"scripts": {
"lint": "tfm-lint",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ exports[`LongDateTime formats date 1`] = `
year="numeric"
>
<span>
October 13, 2017, 11:54 AM
October 13, 2017 at 11:54 AM
</span>
</FormattedDate>
</span>
Expand All @@ -43,7 +43,7 @@ exports[`LongDateTime formats date with relative tooltip 1`] = `
year="numeric"
>
<span>
October 13, 2017, 11:54 AM
October 13, 2017 at 11:54 AM
</span>
</FormattedDate>
</span>
Expand All @@ -68,7 +68,7 @@ exports[`LongDateTime formats date with seconds 1`] = `
year="numeric"
>
<span>
October 13, 2017, 11:54:55 AM
October 13, 2017 at 11:54:55 AM
</span>
</FormattedDate>
</span>
Expand Down

0 comments on commit 39648ff

Please sign in to comment.