refactor: use arrow functions where applicable (#113)

This commit makes use of arrow function expressions wherever applicable. Focus
is put on minimizing the diff instead of optimizing or formatting the code
touched by this commit.

refactor: move CordovaError module up (#120)

Moves the `CordovaError` module from `src/CordovaError/CordovaError.js` to `src/CordovaError.js`. The additional nesting makes no sense anymore since `src/CordovaError/CordovaExternalToolErrorContext.js` has been removed in #117.

refactor(CordovaError)!: remove unused features (#117)

* Remove CordovaExternalToolErrorContext

* Remove unused CordovaError.UNKNOWN_ERROR

* Remove unused verbose mode

* Remove unused getErrorCodeName

* Remove unused support for error codes

* Fix JavaDocs

chore: remove support for ubuntu platform (#111)

chore: consolidate eslint configs (#109)

style: drop jasmine env workaround (#108)

ESLint 6.7.0 has just been released and it contains the latest globals
for the jasmine environment. Thus we do not have to add them ourselves.

refactor: eslint setup (#105)

* replace dependencies with @cordova/eslint-config
* update eslint config
* eslint corrections
* chore: fix newline
* chore: re-add no-unused-vars after-used error
* chore: corrected no-unused-vars settings

test: always run code coverage during `npm test` (#107)

ci(travis): run codecov using npx in after_success (#106)

chore: bump production dependencies (#103)

chore: update jasmine dependencies & config (#101)

chore: replace instanbul w/ nyc (#100)

chore: drop unused & unneeded dependencies (#102)

* chore: drop promise-matchers dependency

* chore: drop osenv dependency

* chore: drop dependency which

chore: improve npm ignore list (#104)

Drop From Package
* all dot file and folders with .* (.github, .travis.yml, .eslintrc.yml, etc...)
  * .github/
  * .travis.yml
  * .eslintrc.yml
  * .eslintignore
  * .ratignore
* appveyor.yml

== PACKAGE REPORT ==
* BEFORE
npm notice package size:  41.5 kB
npm notice unpacked size: 179.4 kB
npm notice total files:   34

* AFTER
npm notice package size:  39.1 kB
npm notice unpacked size: 173.8 kB
npm notice total files:   24

feat: Replace addProperty with ES6 getters (#96)

* Replace addProperty with ES6 getters in main module

This allows static code analysis tools to better understand the module
dependencies. For example, you can use VS Code's code navigation feature
to jump to the module definition.

There's a slight difference in implementation, as the getters installed
by addProperty would only call `require` on first invocation and then
replace themselves with the actual module. But since require does
caching itself, I don't see how that would make a big difference in
terms of runtime.

* Replace addProperty with ES6 getters in ConfigFile

* Remove private module util/addProperty

fix: PluginInfoProvider for scoped plugins (#94)

* fix: PluginInfoProvider for scoped plugins
* Add test for PluginInfoProvider /w scoped plugin
* Normalize paths to use backslash on Windows

chore: update strip-bom@4 (#71)

chore: drop node 6 and 8 support (#90)

Use Array.prototype.find in CordovaError (#97)

Increment package version to 4.0.0-dev

Update version & RELEASENOTES.md for 3.2.1

Re-apply fix for failing CordovaError test (#93)

This fix was first introduced in #65 and reverted by #89

Fixes #49

Remove obsolete JSHint comments (#92)

Convert CordovaError to ES6 class (#87)

(windows) Add .jsproj as file extension for XML files (#62) (#78)

Fixes #62

revert: #24 CB-14108: fix incorrect count in config_munge (#89)

* Revert "CB-14108: fix incorrect count in config_munge in ios.json and android.json"

This reverts commit ce3801a184d34cc4c3ea08aeecd159227c6ae9e9.

* chore: eslint object-curly-spacing fix
* add CordovaError spec
* chore: fix eslint warnings for CordovaError spec
* fix CordovaError spec test
* chore: added additional AS project path check

Merge pull request #82 from brodybits/general-cleanup-with-eslint-update

General cleanup with eslint updates

eslint cleanup fixes for: padded-blocks

and remove padded-blocks exception from .eslintrc.yml

with src & spec fixed by the following command:

    npm run eslint -- --fix

(semi)standard eslint updates

- eslint-config-semistandard@14
- eslint-config-standard@13

with src & spec fixed by the following command:

    npm run eslint -- --fix

eslint cleanup fix for: no-prototype-builtins

as needed for the upcoming (semi)standard eslint package updates

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>
Co-authored-by: Christopher J. Brody <chris@brody.consulting>

eslint cleanup fixes: operator-linebreak (#86)

- remove operator-linebreak exception from .eslintrc.yml
- reformat ternary expresssions according to (semi)standard
- reformat error message expressions according to (semi)standard

as done by the following command, with no operator-linebreak
exception enabled:

    npm run eslint -- --fix

remove no-throw-literal lint exception not needed (#81)

Fix ESLint violations where applicable (#83)

Fixes most ESLint violations that were masked by ESLint comments. Two instances still remain. One is an acceptable usage of `eval` and is in `CordovaError` that should be fixed by its own commit.

Update to jasmine 3.4 & fix resulting spec failures (#80)

Promise handling cleanup in specs (#79)

* Proper expected rejection check in ActionStack spec

* Clean up Promise handling in PluginManager spec

* Remove usage of jasmine's done in sync specs

* Clean up Promise handling in superspawn spec

Do not ignore AppVeyor failures on Node.js 12 (#77)

Increment package version to 3.2.1-dev

Update version & RELEASENOTES.md for 3.2.0

Add Node.js 12 to CI Services (#68)

* Add Node.js 12 to CI Services

* TMP: Allow Node.js 12 Failure on AppVeyor

(ios) Plist document not indented with tabs

Xcode indents .plist documents with tabs, not spaces. Other parts of
Cordova already use tabs for indenting .plist documents and using spaces
directly conflicts with both cases.

Update fs-extra to v8 (#70)

Add example usage of podspec declarations

When trying to find more information about new `<podspec>` tag deprecating `<framework type="podspec" />` I found this example usage.
However it doesn't specify how to use declarations so I've copied example from [specs fixture](https://github.com/apache/cordova-common/blob/rel/3.1.0/spec/fixtures/plugins/org.test.plugins.withcocoapods/plugin.xml#L37-L47)

implement setPreference and setPlatformPreference

Catch leaked exceptions in superspawn and convert them to rejections

At least until Node.js 8, child_process.spawn will throw exceptions
instead of emitting error events in certain cases (like EACCES), Thus we
have to wrap the execution in try/catch to convert them into rejections.

Merge pull request #65 from raphinesse/test-fixes

Test fixes

Fix failing CordovaError test

Fixes #49

Properly handle promise in superspawn test

Add or update GitHub pull request and issue template

Set version to 3.1.1-dev

Update version & RELEASENOTES.md for release 3.1.0

Merge pull request #60 from brodybits/events-singleton

Cordova events singleton object

Update Cordova events to use a real singleton

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Darryl Pogue <darryl@dpogue.ca>

Refactor Cordova events into ES6 class

and replace var with const & let

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Darryl Pogue <darryl@dpogue.ca>

Merge pull request #59 from brodybits/events-spec-updates

Cordova events test updates in spec

Add Cordova events emit method tests to spec

based on existing test code

with other minor Cordova events test updates

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: daserge <v-seshak@microsoft.com>

Explicit Cordova events section in spec

with forwardEventsTo method section indented

Refactor CordovaLogger to singleton class (#53)

Verbose test reporter (#55)

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>
Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>

Cleanup indentation spacing in jasmine.json (#54)

Incremented package version to 3.0.1-dev

Updated version and RELEASENOTES.md for v3.0.0

Fix eslint object-curly-spacing complaints

Update fs-extra and eslint dependencies

Use `cross-spawn` for platform-independent spawning

Closes #50.

add method getPodSpecs and add related tests

CB-13496: Fix greedy regex in plist-helpers

CB-14108: fix incorrect count in config_munge in ios.json and android.json

* Fix to cordova prepare increment count at config_munge in platformJson
* Fix to cordova prepare when remove config-file case

Normalize whitespace (#46)

Fix xml-helpers issue by adding close tag

CB-13685 android: Update ConfigParser for Adaptive Icons

Make PlatformJson.generateMetadata more readable

type attributes for header-file is added

Simplify some helpers in util/xml-helpers (#40)

No functional changes. Just cleanup.

Pin Jasmine@~3.1.0 (#37)

CB-12016: (all) Removed cordova-registry-mapper dependency

CB-14099 osx: Fixed Resolve Config Path for OSX

 This closes #32

.travis.yml add spacing

Increment package version to 3.0.0-dev

Replace shelljs calls with fs-extra & which (#21)

added package-lock.json to .rateignore

CB-14112: Incremented package version to 2.3.0-dev

CB-14112: Updated version and RELEASENOTES.md for release 2.2.2

Use strip-bom to strip BOM

Simplify element removal in ConfigParser

Use .includes and .startsWith instead of .indexOf

Remove obsolete JSCS config

CB-13979: More consistency for config.xml lookups

Add GitHub pull request template

CB-14064: Remove Node 4 from CI matrix

Update dependencies

CB-11691: Fix for modifying binary plists

The bplist-parser module returns the plist dictionary object wrapped in
an array. When making modifications with `<edit-config>` or
`<config-file>`, it would add the changes as properties to the array but
then ignore them when writing out to the filesystem.

CB-13770: Warn when <edit-config> or <config-file> not found

When a file is referenced in <edit-config> or <config-file> is not
found, no warning or error is generated.

This change logs a warning, and includes where the file is actually
expected to be based on the filename provided.

Remove unused arguments and configure linter to report them

Lint sources and specs in one go

Before this change, linting errors in the specs would only be reported
after fixing any linting errors in the sources and running the linter
for a second time.

CB-13471: Fix tests and path issues for Windows

CB-13471 : added unit test for config file provider

Remove outdated README for ConfigParser

The README had completely broken Markdown in some parts.
Furthermore, it was missing a lot of the actually provided methods.
Lastly it was the only one of its kind (sub-README) anyway.

Various fixes in README

CB-13744: Recognize storyboards as XML files

This is to allow customization of the CDVLaunchScreen.storyboard to adjust contentmode and background color in config.xml.

Fix repository URL in package.json