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

CB-13674 Incremented package version to -dev

CB-13674 Updated version and RELEASENOTES.md for release 2.2.1

removed rogue console.log that was making lib tests look fugly, changed CordovaLogger calls to events.emit to match most code. Is CordovaLogger used anywhere?

CB-13599 Incremented package version to -dev

CB-13599 Updated version and RELEASENOTES.md for release 2.2.0

Merge pull request #11 from audreyso/CB-13501

CB-13501 : added node 8 support

CB-13501 : added node 8 support

eslint fix

 This closes #9

CB-13471: File Provider fix belongs in cordova-common until we move this code out

CB-11244 : added unit tests for ConfigFile

 This closes #8

CB-11244: eslint errors for spaces

 This closes #7

CB-11244: Spot fix for cordova-android PR #389

CB-13380 Incremented package version to -dev

CB-13380 Updated version and RELEASENOTES.md for release 2.1.1

CB-13145 : added getFrameworks to unit tests

CB-13145: added variable replacing to framework tag

CB-13231 Incremented package version to -dev

CB-13231 Updated version and RELEASENOTES.md for release 2.1.0

CB-13145: removed uncessary check in getFramework

CB-13145 : added getFrameworks to unit test

CB-13145: added variable replacing to framework tag