You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by ra...@apache.org on 2018/08/28 13:12:53 UTC
[cordova-lib] branch master updated (3994b10 -> 39abbe0)
This is an automated email from the ASF dual-hosted git repository.
raphinesse pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cordova-lib.git.
from 3994b10 Only upload coverage when tests succeed (#668)
add 34af01b Fix lint script and hidden linting errors
add f586c56 Allow rejections w/out reason in tests
add 0d7920b Remove obsolete linter comments
add 6c30de4 Remove unused spy variables in tests
add 27854f3 Follow linting rules instead of ignoring them
new 39abbe0 Merge pull request #669 from raphinesse/linting-improvements
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.eslintignore | 2 +-
cordova-lib.js | 4 ++--
integration-tests/.eslintrc.yml | 4 +++-
integration-tests/platform.spec.js | 2 --
integration-tests/plugman_fetch.spec.js | 8 ++------
integration-tests/plugman_uninstall.spec.js | 28 +++++++-------------------
package.json | 2 +-
spec/.eslintrc.yml | 4 +++-
spec/cordova/build.spec.js | 3 +--
spec/cordova/compile.spec.js | 3 +--
spec/cordova/create.spec.js | 3 ---
spec/cordova/emulate.spec.js | 7 +++----
spec/cordova/metadata/parser.spec.js | 3 +--
spec/cordova/platform/addHelper.spec.js | 1 -
spec/cordova/platform/index.spec.js | 1 -
spec/cordova/platform/platform.spec.js | 1 -
spec/cordova/plugin/add.spec.js | 9 ---------
spec/cordova/plugin/index.spec.js | 6 +-----
spec/cordova/plugin/list.spec.js | 3 ---
spec/cordova/plugin/plugin_spec_parser.spec.js | 1 -
spec/cordova/plugin/save.spec.js | 3 ---
spec/cordova/plugin/search.spec.js | 1 -
spec/cordova/plugin/util.spec.js | 1 -
spec/cordova/plugin_parser.spec.js | 2 +-
spec/cordova/prepare.spec.js | 3 ---
spec/cordova/serve.spec.js | 2 --
spec/cordova/util.spec.js | 4 +---
spec/plugman/add_platform.spec.js | 16 ++-------------
spec/plugman/create.spec.js | 18 +++--------------
spec/plugman/install.spec.js | 24 +++++++---------------
src/cordova/plugin/plugin_spec_parser.js | 2 +-
src/cordova/restore-util.js | 4 ++--
src/hooks/HooksRunner.js | 4 ++--
src/platforms/platforms.js | 4 ++--
src/plugman/install.js | 10 ++++-----
src/plugman/uninstall.js | 6 +++---
src/plugman/util/dependencies.js | 4 ++--
src/plugman/util/strip-license.js | 2 +-
38 files changed, 58 insertions(+), 147 deletions(-)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org
[cordova-lib] 01/01: Merge pull request #669 from
raphinesse/linting-improvements
Posted by ra...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
raphinesse pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cordova-lib.git
commit 39abbe09c1073a76c3db4d90e04b1ff79229ef9f
Merge: 3994b10 27854f3
Author: Raphael von der Grün <ra...@gmail.com>
AuthorDate: Tue Aug 28 15:12:51 2018 +0200
Merge pull request #669 from raphinesse/linting-improvements
Linting improvements
.eslintignore | 2 +-
cordova-lib.js | 4 ++--
integration-tests/.eslintrc.yml | 4 +++-
integration-tests/platform.spec.js | 2 --
integration-tests/plugman_fetch.spec.js | 8 ++------
integration-tests/plugman_uninstall.spec.js | 28 +++++++-------------------
package.json | 2 +-
spec/.eslintrc.yml | 4 +++-
spec/cordova/build.spec.js | 3 +--
spec/cordova/compile.spec.js | 3 +--
spec/cordova/create.spec.js | 3 ---
spec/cordova/emulate.spec.js | 7 +++----
spec/cordova/metadata/parser.spec.js | 3 +--
spec/cordova/platform/addHelper.spec.js | 1 -
spec/cordova/platform/index.spec.js | 1 -
spec/cordova/platform/platform.spec.js | 1 -
spec/cordova/plugin/add.spec.js | 9 ---------
spec/cordova/plugin/index.spec.js | 6 +-----
spec/cordova/plugin/list.spec.js | 3 ---
spec/cordova/plugin/plugin_spec_parser.spec.js | 1 -
spec/cordova/plugin/save.spec.js | 3 ---
spec/cordova/plugin/search.spec.js | 1 -
spec/cordova/plugin/util.spec.js | 1 -
spec/cordova/plugin_parser.spec.js | 2 +-
spec/cordova/prepare.spec.js | 3 ---
spec/cordova/serve.spec.js | 2 --
spec/cordova/util.spec.js | 4 +---
spec/plugman/add_platform.spec.js | 16 ++-------------
spec/plugman/create.spec.js | 18 +++--------------
spec/plugman/install.spec.js | 24 +++++++---------------
src/cordova/plugin/plugin_spec_parser.js | 2 +-
src/cordova/restore-util.js | 4 ++--
src/hooks/HooksRunner.js | 4 ++--
src/platforms/platforms.js | 4 ++--
src/plugman/install.js | 10 ++++-----
src/plugman/uninstall.js | 6 +++---
src/plugman/util/dependencies.js | 4 ++--
src/plugman/util/strip-license.js | 2 +-
38 files changed, 58 insertions(+), 147 deletions(-)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org