Skip to content

Conversation

@patricknicholls
Copy link
Collaborator

No description provided.

Copy link

@cathey191 cathey191 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions with package versions.
But we also need to look into the inputs, enums and guards as they do not seem to be behaving in the way one would expect.

"devDependencies": {
"@angular-devkit/build-angular": "^14.2.10",
"@angular-devkit/build-angular": "^16.2.10",
"@angular-eslint/builder": "14.4.0",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion The Angular-eslint packages should all be upgraded to match the angular version

"ng-packagr": "^16.2.3",
"nyc": "^15.1.0",
"reflect-metadata": "^0.1.13",
"rxjs": "^6.3.3",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion Upgrade to ^7.5.7

"typescript": "4.6.4",
"zone.js": "~0.11.4"
"typescript": "^4.9.3",
"zone.js": "~0.13.3"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question are we able to go to ^0.14.02?

"compilerOptions": {
"outDir": "../../out-tsc/lib",
"declarationMap": true,
"target": "es2020",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question Why are we removing this and not upgrading?

"node_modules/@types"
],
"lib": [
"es2017",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question Why haven't we upgraded this one to ES2022?

@cathey191
Copy link

cathey191 commented Jan 16, 2024

Issue There is another package.json that also needs it's packages upgraded:
projects/ensure/package.json

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.

2 participants