Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-1877

Separation of devDependencies and dependencies in package.json

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2.1
    • Fix Version/s: 3.3.0
    • Component/s: Gulp
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a new project and clone the repository in it.
      2. Run npm install and check that all the dependencies (including gulp, gulp-clean-css, gulp-concat, gulp-rename and gulp-sass) are installed.
      3. Run bower install.
      4. Run gulp. Check that the build files are created in the project.
      Show
      Create a new project and clone the repository in it. Run npm install and check that all the dependencies (including gulp, gulp-clean-css, gulp-concat, gulp-rename and gulp-sass) are installed. Run bower install . Run gulp . Check that the build files are created in the project.
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE
    • Sprint:
      Moodle Mobile 3.3.0

      Description

      Currently there is no clear separation between which packages are under dependencies and devDependencies (see this for a discussion regarding their difference).
      According to my understanding, only the front-end libraries that the front-end client (ionic-bower, and other bower components) goes under the dependencies tag, while all the existing once should be placed under devDependencies.

      All packages should be placed under devDependencies. When MOBILE-1830 is solved, bower packages should be placed under dependencies.

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  26/May/17