-
Notifications
You must be signed in to change notification settings - Fork 154
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
build: migrate to ngx-deploy-npm v8 #1278
base: master
Are you sure you want to change the base?
Conversation
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
"buildTarget": "production" | ||
"distFolderPath": "dist/packages/avatar" | ||
}, | ||
"dependsOn": ["pre-npm-deploy-build-production"] | ||
}, | ||
"pre-npm-deploy-build-production": { | ||
"executor": "nx:run-commands", | ||
"options": { | ||
"command": "nx run avatar:build:production" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that the default mode of the build is production
, so we can drop the new target creation to build in production mode to make it more streamlined.
It would look like the following:
"npm-deploy": {
"executor": "ngx-deploy-npm:deploy",
"options": {
"access": "public",
"buildTarget": "production"
"distFolderPath": "dist/packages/badge"
},
"dependsOn": ["build"]
}
ngx-deploy-npm introduces some breaking changes on version 8.0.0.
More information about the changes and their motivation can be found on this blog post