rajaramtt / ng7-dynamic-breadcrumb

Angular 10 Dynamic Breadcrumbs
MIT License
42 stars 31 forks source link

Second attempt at ng16 upgrade #51

Closed bjorn-einar-bjartnes-4ss closed 10 months ago

bjorn-einar-bjartnes-4ss commented 1 year ago

Tried a different way to upgrade using ng update. Still does not work, but likely a better way to get dependencies correct. Commands are in individual commits, but I ran these

ng update @angular/animations@15 @angular/cdk@15 @angular/common@15 @angular/compiler@15 @angular/core@15 @angular/forms@15 @angular/material@15 @angular/platform-browser@15 @angular/platform-browser-dynamic@15 @angular/router@15

 ng update @angular/animations@16 @angular/cdk@16 @angular/common@16 @angular/compiler@16 @angular/core@16 @angular/forms@16 @angular/material@16 @angular/platform-browser@16 @angular/platform-browser-dynamic@16 @angular/router@16 @angular/compiler-cli@16 @angular/language-service@16 @angular-devkit/build-angular@16

Realized later that I had forgot ng update @angular/cli@15 and ng update @angular/cli@16 so did that in a separate commit.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ng7-dynamic-breadcrumb ❌ Failed (Inspect) Aug 4, 2023 0:40am
bjorn-einar-bjartnes-4ss commented 1 year ago

By building invidiual components, all four , using ng build I was able to run ng test but it claims some errors, while also being green

image
bjorn-einar-bjartnes-4ss commented 10 months ago

We rewrote our components, I have no further interest in pursuing this stale PR so closing it.