Problem/Motivation

Project should indicate which version(s) of node are supported for each release.

Steps to reproduce

Attempt to run npm install with the incorrect version. Errors will be displayed in terminal, until user upgrades/downgrades to correct node version.

Proposed resolution

There are multiple options for indicating which version(s) of node should be used for a release (even if we're not enforcing it programmatically):

Remaining tasks

Choose one of the options above, and update the codebase.

User interface changes

No UI changes required.

API changes

No API changes.

Data model changes

No Data model changes.

Thank you very much for creating this starter kit!!

CommentFileSizeAuthor
#2 3199572-2.patch508 bytesPooja Ganjage
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zonesny created an issue. See original summary.

Pooja Ganjage’s picture

FileSize
508 bytes

Hi,

Creating a patch for this issue.

Please review the patch.

Let me know if any suggestions.

Thanks.

Pooja Ganjage’s picture

Status: Active » Needs review
candelas’s picture

Thanks #3. I was having problems in the installation with many errors and the node version resolved it.

segx’s picture

This resolved my issue. Thank you!

candelas’s picture

Status: Needs review » Reviewed & tested by the community
griz’s picture

Priority: Normal » Major

Just wasted an hour trying to get this working, before realising I needed to downgrade node.

hussainweb’s picture

While this works, it doesn't feel like a good solution because we are now limiting only to outdated releases of Node. Node 12 and 14 are in maintenance modes only and they are the only supported ones allowed by the version constraint.

I hope we can also get #3259490: Errors on running npm install in subtheme on Node 16+ in so that we can improve the supported versions. For example, I am using this with Node 17 (and 16) without any errors.

emilcarpenter’s picture

Version: 5.0.2 » 5.0.5
Component: Documentation » Code
Category: Task » Bug report
Status: Reviewed & tested by the community » Needs review

@griz no need to downgrade Node.

I got node install working very easily with Node v16 with @hussainweb solution:
https://www.drupal.org/project/bootstrap_sass/issues/3259490#comment-14379708

emilcarpenter’s picture

griz’s picture

Ah, amazing - thanks!

hatuhay’s picture

Status: Needs review » Closed (outdated)