[ofbiz-framework] branch trunk updated: Improved: (OFBIZ-)

[jl...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 32d0a51  Improved: (OFBIZ-)
32d0a51 is described below

commit 32d0a51db04696712e35512255fc40cc33be9c90
Author: Jacques Le Roux <ja...@les7arts.com>
AuthorDate: Wed Oct 27 18:02:35 2021 +0200

    Improved:
    (OFBIZ-)
    
    Following
    https://github.com/node-gradle/gradle-node-plugin/blob/master/docs/faq.md#how-do-i-use-npm-ci-instead-of-npm-install
    
    I have tried the feature (see in the Jira issue).
    I'm not sure it's currently quite useful in OFBiz.
    Anyway it does not hurt to add this possibility, notably for specific CI cases
    
    Note that when updating Gradle we may need to review that:
    https://github.com/node-gradle/gradle-node-plugin/blob/master/docs/faq.md#is-this-plugin-compatible-with-centralized-repositories-declaration
---
 build.gradle | 1 +
 1 file changed, 1 insertion(+)

diff --git a/build.gradle b/build.gradle
index a48cb3d..64843e9 100644
--- a/build.gradle
+++ b/build.gradle
@@ -115,6 +115,7 @@ node {
     // https://github.com/node-gradle/gradle-node-plugin/blob/2.2.4/README.md
     //// Set the work directory where node_modules should be located
     nodeModulesDir = file("${project.projectDir}/themes/common-theme/webapp/common/js")
+    npmInstallCommand = System.getenv("CI") ? 'ci' : 'install'
 }