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

Add cordovaBasePath to overrides options #57

Merged
merged 7 commits into from
Jan 28, 2019
Merged

Add cordovaBasePath to overrides options #57

merged 7 commits into from
Jan 28, 2019

Conversation

GuilhermeBCC
Copy link
Contributor

The cordovaBasePath property is required for Monorepo projects, since the applications are not in the project root.

The cordovaBasePath property is required for Monorepo projects, since the applications are not in the project root.
@imhoffd
Copy link
Contributor

imhoffd commented Jan 16, 2019

@GuilhermeBCC Good catch! Want to just run npm run lint -- --fix and commit the changes?

@imhoffd imhoffd merged commit 93e3bbe into ionic-team:master Jan 28, 2019
@imhoffd
Copy link
Contributor

imhoffd commented Jan 28, 2019

Thank you!

Ionitron added a commit that referenced this pull request Jan 29, 2019
# [1.3.0](v1.2.3...v1.3.0) (2019-01-29)

### Bug Fixes

* **serve:** pass `cordovaBasePath` to cordova-build builder ([#57](#57)) ([93e3bbe](93e3bbe))

### Features

* **build:** add `--cordova-mock` option ([#63](#63)) ([a659636](a659636))
@Ionitron
Copy link
Collaborator

🎉 This PR is included in version 1.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

wand1252 added a commit to wand1252/angular-toolkit-develop that referenced this pull request Aug 31, 2022
# [1.3.0](ionic-team/angular-toolkit@v1.2.3...v1.3.0) (2019-01-29)

### Bug Fixes

* **serve:** pass `cordovaBasePath` to cordova-build builder ([#57](ionic-team/angular-toolkit#57)) ([93e3bbe](ionic-team/angular-toolkit@93e3bbe))

### Features

* **build:** add `--cordova-mock` option ([#63](ionic-team/angular-toolkit#63)) ([a659636](ionic-team/angular-toolkit@a659636))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants