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

fix: save package in devDependencies #81

Merged
merged 1 commit into from
Nov 6, 2019

Conversation

alan-agius4
Copy link
Contributor

In the latest versions of the CLI ng-add packages can be added to devDependencies and this package is perfect for such use case since it's only needed for development.

See: angular/angular-cli#15815

In the latest versions of the CLI `ng-add` packages can be added to `devDependencies` and this package is perfect for such use case since it's only needed for development.

See: angular/angular-cli#15815
@fmalcher
Copy link
Member

fmalcher commented Nov 6, 2019

LGTM
Thank you, Alan! @alan-agius4

@JohannesHoppe
Copy link
Member

Thank you very much for pointing this out. I will add this to angular-cli-ghpages and ngx-deploy-starter!

@santoshyadavdev
Copy link
Contributor

Thank you very much for pointing this out. I will add this to angular-cli-ghpages and ngx-deploy-starter!

He already raised a PR :)

@JohannesHoppe JohannesHoppe merged commit 3dfd8d6 into angular-schule:master Nov 6, 2019
@alan-agius4 alan-agius4 deleted the patch-1 branch November 6, 2019 12:53
JohannesHoppe pushed a commit to angular-schule/ngx-deploy-starter that referenced this pull request Nov 6, 2019
In the latest versions of the CLI `ng-add` packages can be added to `devDependencies` and this package is perfect for such use case since it's only needed for development.

See: angular/angular-cli#15815
(cherry picked from commit 3dfd8d6)

closes #12
see angular-schule/angular-cli-ghpages#81
xellafe pushed a commit to xellafe/ngx-deploy-git-repo that referenced this pull request Oct 2, 2024
In the latest versions of the CLI `ng-add` packages can be added to `devDependencies` and this package is perfect for such use case since it's only needed for development.

See: angular/angular-cli#15815
(cherry picked from commit 3dfd8d64a6781a4081816a2022a66283c2b7247b)

closes #12
see angular-schule/angular-cli-ghpages#81
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants