Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cordova Lib Release Preparation (Cordova 9) #749

Merged
merged 15 commits into from Mar 14, 2019
Merged

Conversation

erisu
Copy link
Member

@erisu erisu commented Mar 13, 2019

Platforms affected

all

Motivation and Context

Cordova 9 Release

Description

This PR contains final preparations for the Cordova 9 release goals.

See Cordova 9 Release Plan.

  • Bumped Dependencies
    • cordova-common@^3.1.0
    • cordova-create@^2.0.0
    • cordova-fetch@^2.0.0
    • cordova-serve@^3.0.0
    • fs-extra@^7.0.1
    • globby@^9.1.0
    • init-package-json@^1.10.3
    • read-chunk@^3.1.0
    • semver@^5.6.0
    • underscore@^1.9.1
  • Bumped ESLint dependencies with correction
    • eslint@^5.15.1
    • eslint-config-semistandard@^13.0.0
    • eslint-config-standard@^12.0.0
    • eslint-plugin-import@^2.16.0
    • eslint-plugin-node@^8.0.1
    • eslint-plugin-promise@^4.0.1
    • eslint-plugin-standard@^4.0.0
  • Updated Dev Dependencies
    • Bumped codecov@^3.2.0
    • Bumped delay@^4.1.0
    • Bumped jasmine@^3.3.1
    • Bumped nyc@^13.3.0
    • Added shelljs@^0.8.3

Platform Pinning Bump
All platform pinnings were bumped and increase semver matches to include minors.

Testing

Checklist

  • I've run the tests to see all new and existing tests pass
  • I added automated test coverage as appropriate for this change
  • Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
  • If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
  • I've updated the documentation if necessary

@erisu erisu requested review from raphinesse and dpogue March 13, 2019 06:46
@project-bot project-bot bot added this to 🐣 New PR / Untriaged in Apache Cordova: Tooling Pull Requests Mar 13, 2019
erisu and others added 2 commits March 13, 2019 16:10
npm >= 5 automatically prunes any unsaved packages during installation
(see npm/npm#16853). Therefore, installing
platforms and plugins without saving them to package.json is essentially
broken right now.
@codecov-io
Copy link

codecov-io commented Mar 13, 2019

Codecov Report

Merging #749 into master will not change coverage.
The diff coverage is 82.35%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #749   +/-   ##
=======================================
  Coverage   86.04%   86.04%           
=======================================
  Files          50       50           
  Lines        2666     2666           
=======================================
  Hits         2294     2294           
  Misses        372      372
Impacted Files Coverage Δ
src/platforms/platforms.js 100% <ø> (ø) ⬆️
src/plugman/platform.js 20% <0%> (ø) ⬆️
src/cordova/config.js 46.34% <0%> (ø) ⬆️
src/cordova/prepare.js 100% <100%> (ø) ⬆️
src/cordova/serve.js 100% <100%> (ø) ⬆️
src/plugman/create.js 91.42% <100%> (ø) ⬆️
src/cordova/restore-util.js 95.26% <100%> (ø) ⬆️
src/cordova/plugin/add.js 98.27% <100%> (ø) ⬆️
src/hooks/HooksRunner.js 84.68% <100%> (ø) ⬆️
src/cordova/plugin/save.js 98.46% <100%> (ø) ⬆️
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 65ca227...8b4896f. Read the comment docs.

Copy link
Contributor

@raphinesse raphinesse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Added two commits with minor fixes. Feel free to move them to separate PRs if preferred.

Apache Cordova: Tooling Pull Requests automation moved this from 🐣 New PR / Untriaged to ✅ Approved, waiting for Merge Mar 13, 2019
@erisu
Copy link
Member Author

erisu commented Mar 14, 2019

Thanks @raphinesse !

I will be merging this PR and not squashing. I can keep these commits under this one PR.

@erisu erisu merged commit 733b3a4 into apache:master Mar 14, 2019
Apache Cordova: Tooling Pull Requests automation moved this from ✅ Approved, waiting for Merge to 🏆 Merged, waiting for Release Mar 14, 2019
@erisu erisu deleted the cordova-9-prep branch April 4, 2019 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Apache Cordova: Tooling Pull Requests
🏆 Merged, waiting for Release
Development

Successfully merging this pull request may close these issues.

None yet

3 participants