Is this a good idea

doing_a_build_involving_npm.sh

npm_install()
{
  cd $PROJECT_HOME   # package.json in here
  if npm install
  then
      return 0
  else
      return 1
  fi
}

do_build()
{

  if ! type npm > /dev/null
  then
    echo "npm command not found. Please install npm and/or configure your shell."
    exit 1
  fi

  if npm_install    # call my function
  then
    # foo
    echo "npm install was cool"
  else
    # fail
    echo "npm install is mega fail"
  fi
}