You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by GitBox <gi...@apache.org> on 2021/08/19 03:50:28 UTC

[GitHub] [cordova-docs] erisu opened a new pull request #1189: chore!: update all npm dependencies

erisu opened a new pull request #1189:
URL: https://github.com/apache/cordova-docs/pull/1189


   ### Motivation and Context
   
   - Keep up-to-date with the npm packages
   - Try to remove as many NPM audit warnings
   
   ### Description
   
   #### Bumped npm packages:
   
   **Major/Breaking**
   
   - `babelify@10.0.0`
   - `baconjs@3.0.17`
   - `browserify@17.0.0`
   - `chalk@4.1.2`
   - `envify@4.1.0`
   - `fs-extra@10.0.0`
   - `gulp-footer@2.0.2`
   - `gulp-uglify@3.0.2`
   - `gulp-header@2.0.9`
   - `gulp-replace@1.1.3`
   - `gulp-less@5.0.0`
   - `gulp-rename@2.0.0`
   - `gulp-sass@5.0.0`
   - `gulp-htmllint@0.0.19`
   - `js-yaml@4.1.0`
   - `simplecrawler@1.1.9`
   
   **Minor/Patch**
   
   - `classnames@2.3.1`
   - `glob@7.1.7`
   - `gulp-util@3.0.8`
   - `semver@7.3.5`
   - `ncp@2.0.0`
   - `optimist@0.6.1`
   - `browser-sync@2.27.5`
   - `vinyl-source-stream@2.0.0`
   - `vinyl-buffer@1.0.1`
   
   #### Removed npm packages:
   
   - `babel-plugin-transform-h-jsx`
   - `babel-plugin-transform-react-jsx`
   - `babel-preset-react`
   - `eslint`
   - `eslint-config-semistandard`
   - `eslint-config-standard`
   - `eslint-plugin-import`
   - `eslint-plugin-node`
   - `eslint-plugin-promise`
   - `eslint-plugin-standard`
   
   #### Added npm packages:
   
   - `@babel/core@7.15.0`
   - `@babel/preset-react@7.14.5`
   - `@cordova/eslint-config@3.0.0`
   - `sass@1.38.0`
   
   #### Refactored:
   
   **Packages Moved to `devDependencies`**
   
   - `browser-sync`
   - `browserify`
   - `chalk`
   - `envify`
   - `fs-extra`
   - `gulp-footer`
   - `gulp-uglify`
   - `gulp`
   - `gulp-header`
   - `gulp-replace`
   - `gulp-less`
   - `gulp-rename`
   - `gulp-sass`
   - `gulp-htmllint`
   - `js-yaml`
   - `ncp`
   - `optimist`
   - `simplecrawler`
   - `vinyl-source-stream`
   - `vinyl-buffer`
   
   **Others:**
   
   - Applied safe schema to all `js-yaml` usage.
   - ESLint
     - Updated ESLint configs
     - Applied mutomatic lint corrections
     - Applied manual correction: thrown error to throw an Error object.
   - Rebuilt the `package-lock.json`
   - Rename `npm run eslint` command to `npm run lint`
   
   ### Testing
   
   - `npm t`
   - `npm run build`
   - `npm run build:prod`
   - `npm run lint`
   
   - I compared the before and after prod builds: https://github.com/apache/cordova-docs/commit/0651c15d4cf5d81478a42181a7bc470574bd1f11
   
   ### Checklist
   
   - [x] I've run the tests to see all new and existing tests pass
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org


[GitHub] [cordova-docs] erisu merged pull request #1189: chore!: update all npm dependencies w/ minor corrections

Posted by GitBox <gi...@apache.org>.
erisu merged pull request #1189:
URL: https://github.com/apache/cordova-docs/pull/1189


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org