Skip to content

fix: save package in devDependencies #15

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

Merged
merged 1 commit into from
Nov 7, 2019

Conversation

alan-agius4
Copy link

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
@dianjuar
Copy link
Member

dianjuar commented Nov 7, 2019

I saw that you are updating all the deployers 😄

Would be nice to have this fix on the original repo, that one that I used to create this deployer ngx-deploy-starter

(Edit)
You already did it 😅

@dianjuar dianjuar self-assigned this Nov 7, 2019
@dianjuar
Copy link
Member

dianjuar commented Nov 7, 2019

I will run the unit and manual test to see if everything is Ok and then integrate it.

Thank you so much for your contribution

@JohannesHoppe
Copy link

👍

@dianjuar dianjuar merged commit 877c281 into bikecoders:master Nov 7, 2019
@alan-agius4 alan-agius4 deleted the patch-1 branch November 7, 2019 16:32
@dianjuar
Copy link
Member

dianjuar commented Nov 7, 2019

@alan-agius4 since which version of the Angular/Cli this feature is available ?

@alan-agius4
Copy link
Author

v9.0.0-next.10

dianjuar pushed a commit that referenced this pull request Jan 26, 2020
…c/handlebars-4.5.3

build(deps): bump handlebars from 4.1.2 to 4.5.3 in /src
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.

3 participants