Skip to content

Commit

Permalink
fix: ng update showing packages that do not exist in my package
Browse files Browse the repository at this point in the history
By default if no `packageGroupName` is provided `ng-update` will use the first item in `packageGroup` which in this case it is always `@nguniversal/aspnetcore-engine`.

With this change we set the `packageGroupName` of each package, also we remove `ng-update` from `@nguniversal/builders` as this gets updated when running `ng update` on `@nguniversal/express-engine`  or `@nguniversal/hapi-engine`.

Closes #1542 and closes #1533
  • Loading branch information
alan-agius4 committed Mar 9, 2020
1 parent db6c9f2 commit 08657f9
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 8 deletions.
3 changes: 2 additions & 1 deletion modules/aspnetcore-engine/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@
"tslib": "TSLIB_VERSION"
},
"ng-update": {
"packageGroup": "NG_UPDATE_PACKAGE_GROUP"
"packageGroup": "NG_UPDATE_PACKAGE_GROUP",
"packageGroupName": "@nguniversal/aspnetcore-engine"
},
"repository": {
"type": "git",
Expand Down
3 changes: 0 additions & 3 deletions modules/builders/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,5 @@
"rxjs": "RXJS_VERSION",
"tree-kill": "^1.2.1",
"guess-parser": "^0.4.12"
},
"ng-update": {
"packageGroup": "NG_UPDATE_PACKAGE_GROUP"
}
}
3 changes: 2 additions & 1 deletion modules/common/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
"tslib": "TSLIB_VERSION"
},
"ng-update": {
"packageGroup": "NG_UPDATE_PACKAGE_GROUP"
"packageGroup": "NG_UPDATE_PACKAGE_GROUP",
"packageGroupName": "@nguniversal/common"
},
"repository": {
"type": "git",
Expand Down
3 changes: 2 additions & 1 deletion modules/express-engine/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@
"schematics": "./schematics/collection.json",
"ng-update": {
"migrations": "./schematics/migrations/migration-collection.json",
"packageGroup": "NG_UPDATE_PACKAGE_GROUP"
"packageGroup": "NG_UPDATE_PACKAGE_GROUP",
"packageGroupName": "@nguniversal/express-engine"
},
"repository": {
"type": "git",
Expand Down
3 changes: 2 additions & 1 deletion modules/hapi-engine/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,8 @@
"schematics": "./schematics/collection.json",
"ng-update": {
"migrations": "./schematics/migrations/migration-collection.json",
"packageGroup": "NG_UPDATE_PACKAGE_GROUP"
"packageGroup": "NG_UPDATE_PACKAGE_GROUP",
"packageGroupName": "@nguniversal/hapi-engine"
},
"repository": {
"type": "git",
Expand Down
3 changes: 2 additions & 1 deletion modules/socket-engine/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@
"tslib": "TSLIB_VERSION"
},
"ng-update": {
"packageGroup": "NG_UPDATE_PACKAGE_GROUP"
"packageGroup": "NG_UPDATE_PACKAGE_GROUP",
"packageGroupName": "@nguniversal/socket-engine"
},
"repository": {
"type": "git",
Expand Down

0 comments on commit 08657f9

Please sign in to comment.