Andrew Kushnir

Results 312 comments of Andrew Kushnir

[Presubmit](http://test/OCL:764733590:BASE:764729850:1748531413149:be41c2d6).

@JeanMeche in the [current version](https://angular.dev/tutorials/learn-angular/20-inject-based-di) of the tutorial, there is no TS error that I've mentioned above (the field access is inside of a constructor function). Can we resolve this...

> Fwiw, starting the docs from the main branch exposes the same issue. So this is something pre-existing. @JeanMeche should we proceed with the merge (and create a new ticket)...

@arturovt thanks for the improvement 👍 The change looks good, but I'm less familiar with this part of the codebase, so I'm adding @alxhub as a reviewer as well.

> Worth mentionning, should we also change : @JeanMeche yes, let's change those instances as well 👍

This PR was merged into the repository by commit 9b96fccbc442e33add973915f8cc930a09e3ce90. The changes were merged into the following branches: main, 19.2.x, 20.0.x

@josephperrott should we bump the versions to rc.3 (once CLI is released)?

@devversion thanks for the review 👍 > For the deps cleanup, I wonder if we will actually see some improvements there, as the lock files weren't updated for a few;...

**Caretaker note**: the `lint` failure is a known behavior of the tool, not caused by this PR (this PR doesn't bring new dependencies). This PR is ready for merge.

[Presubmit](http://test/OCL:708434351:BASE:708433910:1734735063126:4b372ae6).