main
Name already in use
Commits on Apr 6, 2023
-
ci: remove lock-closed action (#49650)
Remove the lock-closed asction as it has been centralized within dev-infra PR Close #49650
-
-
refactor(core): update the link in hydration stats message (#49728)
Currently, the link points to https://angular.io/guides/hydration and there are 2 issues with it: the `guides/hydration` should actually be `guide/hydration` and the guide is only available at https://next.angular.io, but not at https://angular.io. It will be available at https://angular.io once v16 final is released. For now, we can point to https://next.angular.io, so that developers testing hydration during the pre-release period can follow the link. PR Close #49728
-
refactor(core): drop a reference to
enableProdMode()
in a console.l… -
refactor(core): skip hydration for components that use ShadowDom enca…
-
refactor(core): skip hydration for components that use i18n (instead …
-
-
-
-
Commits on Apr 5, 2023
-
refactor(platform-browser): remove deprecated `BrowserTransferStateMo…
…dule` symbol (#49718) The `TransferState` class was updated in v14 in a way that the `BrowserTransferStateModule` was no longer required. The `BrowserTransferStateModule` was deprecated in v14.1 and it's currently empty. This commit removes the `BrowserTransferStateModule`. BREAKING CHANGE: The deprecated `BrowserTransferStateModule` was removed, since it's no longer needed. The `TransferState` class can be injected without providing the module. The `BrowserTransferStateModule` was empty starting from v14 and you can just remove the reference to that module from your applications. PR Close #49718
-
refactor(animations): remove Domino specific logic (#49630)
These workarounds are no longer required following the fixes in angular/domino#14 PR Close #49630
-
refactor(platform-server): remove
ServerRendererFactory2
and `Emula……tedEncapsulationServerRenderer2` (#49630) These methods are no longer required following the fixes in angular/domino#14 PR Close #49630
-
refactor(platform-browser): update renderer to be able to remove `Ser…
…verRendererFactory2` and `EmulatedEncapsulationServerRenderer2` (#49630) This commits updates the render to able to handle the slight differences between platform-server and platform-browser. This is needed to eventually be able to remove `ServerRendererFactory2` and `EmulatedEncapsulationServerRenderer2` from platform-server. PR Close #49630
-
refactor(core): add explicit import for ngDevMode (#49708)
This commit adds an explicit import of the ngDevMode definition to the files in the signal package which use it. PR Close #49708
-
refactor(core): export additional symbols from the signals index (#49708
) This commit adds a few missing exports from the signals index. PR Close #49708
-
refactor(core): add
postSignalSetFn
hook forWritableSignal
(#49708)This commit adds a hook to `WritableSignal` that is called whenever the signal's value is updated via the mutation API. This hook allows consumers to implement logic which is synchronous with signal sets (e.g. executing effects). It's currently unused. PR Close #49708
-
build: combine
@bazel/concatjs
patches (#49714)This commit combines all the `@bazel/concatjs` patches PR Close #49714
-
build: update domino digest to aa8de34 (#49716)
See associated pull request for more information. PR Close #49716
-
fix(core): make sure that lifecycle hooks are not tracked (#49701)
Angular lifecycle hooks should never be run as part of the reactive context: we do not expect that signal reads in lifecycle hooks report to any consumers. In the current Angular some of the lifecycle hooks can be flushed early, while executting template update pass. We need to make sure that signal reads in those lifecycle hooks do not register as part of the effect that marks components for check. " PR Close #49701
-
fix(core): When using setInput, mark view dirty in same was as `markF…
…orCheck` (#49711) `ComponentRef.setInput` internally calls `markDirtyIfOnPush` which only marks the given view as dirty but does not mark parents dirty like `ChangeDetectorRef.markForCheck` would. https://github.com/angular/angular/blob/f071224720f8affb97fd32fb5aeaa13155b13693/packages/core/src/render3/instructions/shared.ts#L1018-L1024 `markDirtyIfOnPush` has an assumption that it’s being called from the parent’s template. That is, we don’t need to mark dirty to the root, because we’ve already traversed down to it. The function used to only be called during template execution for input bindings but was added to `setInput` later. It's not a good fit because it means that if you are responding to events such as an emit from an `Observable` and call `setInput`, the view of your `ComponentRef` won't necessarily get checked when change detection runs next. If this lives inside some `OnPush` component tree that's not already dirty, it only gets refreshed if you also call `ChangeDetectorRef.markForCheck` in the host component (because it will be "shielded" be a non-dirty parent). PR Close #49711
-
feat(core): support usage of non-experimental decorators with TypeScr…
…ipt 5.0 (#49492) Previously, attempting to turn off the `experimentalDecorators` TypeScript configuration option within an Angular project would result in build time errors. These errors were due to an exposed Decorator signature from `@angular/core` that TypeScript thought was incompatible with standard decorators. However, Angular's class decorators (`Component`, `Directive`, `Pipe`, `Injectable`, `NgModule`) are actually already compatible with standard decorators. The export types for the decorators only needed to be updated to reflect that compatibility. With the updated exported types, applications will now successfully compile and execute in AOT mode with one important dependency injection caveat explained in the note below. For JIT mode applications that are built with the Angular CLI, `@ngtools/webpack`, or use `tsickle`, there were also no additional changes required. These tools automatically convert property decorators (now called field decorators) at build time to store Angular property metadata directly on the relevant class. Building with these tools is the overwhelmingly common method of building an application. Any applications that do not use one of these tools will not function at runtime in JIT mode if using standard decorators. The behavior and code for when experimental decorators is enabled has been left intact. NOTE: Angular constructor dependency injection that requires parameter decorators is not supported. The standard decorator specification does not support parameter decorators. The `inject` function must be used for all cases that previously required a parameter decorator. This includes such decorators as `Inject`, `Optional`, `Self`, `SkipSelf`, `Host`, and `Attribute`. Constructor dependency injection that relies only on the supplied parameter type will continue to function as expected if using AOT; as well as in JIT mode if using the Angular CLI, `@ngtools/webpack` directly, or `tsickle`. Documentation for the `inject` function can be found at: https://angular.io/api/core/inject The decorator specification proposal can be found at: https://github.com/tc39/proposal-decorators PR Close #49492
-
refactor(forms): Add a test that uses ControlConfig with a non-static…
Commits on Apr 4, 2023
-
refactor(core): Remove
ReflectiveInjector
symbol (#48103)The `ReflectiveInjector` symbol has been deprecated in v5 (11 major versions ago). This commit removes ReflectiveInjector and related symbols. BREAKING CHANGE: The `ReflectiveInjector` and related symbols were removed. Please update the code to avoid references to the `ReflectiveInjector` symbol. Use `Injector.create` as a replacement to create an injector instead. PR Close #48103
-
build: update eslint dependencies to v5.57.1 (#49696)
See associated pull request for more information. PR Close #49696
-
refactor(core): Cleanup for Iterable differs test. (#49598)
Remove some `any` and use the non-deprecated `Injector.create` overload. PR Close #49598
-
feat(core): Drop public
factories
property forIterableDiffers
: … -
refactor(router): Remove RouterTestingModule in favor of RouterModule…
….forRoot (#49427) `RouterTestingModule` is not needed as of v16. Instead, TestBed automatically provides `MockPlatformLocation` in order to help test navigations in the application. The location mocks in the RouterTestingModule aren't necessary anymore. There doesn't appear to be any real documentation around `RouterTestingModule` other than the API docs. PR Close #49427
-
refactor(core): remove
ɵmakeDecorator
from private exports (#49300)`makeDecorator` is unused outside of core and is not part of the public API (not exported). PR Close #49300
-
fix(router): create correct URL relative to path with empty child (#4…
-
build: update domino digest to 89bec1a (#49703)
See associated pull request for more information. PR Close #49703