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
Feature/angular15 material legacy #3774
Feature/angular15 material legacy #3774
Conversation
17d666a
to
fcf4351
Compare
6d9ccfc
to
ec5a3f9
Compare
ec5a3f9
to
fcf23f9
Compare
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.
Small changes to apply
@@ -15,39 +15,39 @@ | |||
"url": "https://github.com/nationalbankbelgium/stark.git" | |||
}, | |||
"engines": { | |||
"node": ">=14.20.0", | |||
"npm": ">=8.19.1" | |||
"node": ">=18.17.1", |
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.
Please adapt the value in the ".nvmrc" file:
18.17.1
package.json
Outdated
@@ -39,13 +39,13 @@ | |||
"@commitlint/config-conventional": "^18.6.0", | |||
"@compodoc/compodoc": "1.1.19", | |||
"@mdi/angular-material": "^4.0.96", | |||
"@nationalbankbelgium/code-style": "^1.9.0", | |||
"@nationalbankbelgium/code-style": "file:../code-style/dist/nationalbankbelgium-code-style-1.9.0.tgz", |
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.
Please revert this change
tsconfig.json
Outdated
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.
Please remove this file
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.
Please update the dependencies for each stark module then commit the changes in the main "package.json" at the same time.
For instance:
- Update the dependency "webpack-bundle-analyzer" to version "^4.10.1" in the main "package.json" file
- Update the dependency "webpack-bundle-analyzer" to version "^4.10.1" in the packages/stark-build/package.json file
- Commit the changes in the 3 files (2 package.json + 1 package-lock.json) and mention the updated library:
It could be a message like:feat(stark-build): update `webpack-bundle-analyzer` dependency to version `^4.10.1`
fcf23f9
to
3610514
Compare
ecdf057
to
c35d150
Compare
… `@uirouter/angular` dependencies Update node version in `.nvmrc`.
Run schematics to update to angular 15: `npx ng update @angular/core@15 @angular/cli@15 @angular/material@15`
Sync dependencies with root `package.json`. Update following dependencies: - `@angular-builders/custom-webpack` : `^15.0.0` - `webpack-bundle-analyzer` : `^4.10.1`
Sync dependencies with root `package.json`. Update following dependencies: - `@ng-idle/core`: `^13.0.1` - `@ng-idle/keepalive`: `^13.0.1` - `@ngrx/effects`: `^15.4.0` - `@ngrx/store`: `^15.4.0` - `@ngrx/store-devtools`: `^15.4.0` - `@ngx-translate/core`: `^14.0.0` - `@uirouter/angular`: `^11.1.0` Refactor `StarkSettingsEffects` class due to change in `typescript` configuration: - `"useDefineForClassFields": true`
Sync dependencies with root `package.json`. Update following dependency: - `@angular/material-moment-adapter`: `^15.2.9` Refactor Effects classes due to change in `typescript` configuration: - `"useDefineForClassFields": true`
Sync dependencies with root `package.json`. Update following dependencies: - `@angular-devkit/build-angular`: `^15.2.10` - `puppeteer`: `^22.4.0`
Refactor Effects classes due to change in `typescript` configuration: - `"useDefineForClassFields": true`
c35d150
to
ee7237e
Compare
Quality Gate passedIssues Measures |
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.
LGTM 👍
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Breaking change related to angular 15 and @angular/material@15
Other information