Conversation
|
Hi @ramijebara
Thank you, |
|
Thank you @pavandixit93! I will review today. |
ramijebara
left a comment
There was a problem hiding this comment.
Looks good so far, Thank you!
I have added a couple of comments and suggestions for you. Please take a look and update the PR as needed.
Feature/angular 15 npm audit
|
@pavandixit93 Thank you! I noticed you made some changes in response to the comments. Please give me a chance to review and let you know. |
|
@pavandixit93 I will accept and merge your PR. Thank you. Please note that after the PR is merged I will make some minor changes to remove code and config that is no longer relevant. Once done, I will push the new package to npm. |
|
@ramijebara Great, Thank you. Let me know once package is published, i will start using the same in my project. Thanks for your help. |
|
@pavandixit93 the npm package was published earlier today. Thank you! |
|
@ramijebara Thank you so much. |
Angular migration:
cexbrayat/angular-cli-diff@14.2.0...15.1.4
LINTING:

FRONTEND CODE COVERAGE:
Tests configuration is broken in the main branch and also in this fork.
There are currently no test files in the entire project.
Builds successfully:

Outputs correct assets to dist:

Working demo:
angular2gridster-working-demo.webm