alleyinteractive / byline-manager

Manage an article's byline and author profiles in WordPress.
GNU General Public License v3.0
3 stars 2 forks source link

Mismatched Node requirements #237

Closed nexstar-tom-j-nowell closed 7 months ago

nexstar-tom-j-nowell commented 8 months ago

Description of the bug

bylines manager requires Node v16, but its dependencies require Node 18/20!

Steps To Reproduce

❯ npm ci
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'postcss-focus@7.0.0',
npm WARN EBADENGINE   required: { node: '^18.0.0 || >=20.0.0' },
npm WARN EBADENGINE   current: { node: 'v16.20.0', npm: '8.19.4' }
npm WARN EBADENGINE }
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'postcss-import@16.0.0',
npm WARN EBADENGINE   required: { node: '>=18.0.0' },
npm WARN EBADENGINE   current: { node: 'v16.20.0', npm: '8.19.4' }
npm WARN EBADENGINE }

Acceptance Criteria

Additional Information

I used nvm use to make sure I had the appropriate Node version installed, then ran npm ci and saw the issue.

I would recommend upgrading to Node 20 as that's what core uses, 18 at a minimum

kaitlinbolling commented 7 months ago

Updated in version 2.0.5.