forked from angular/angular
-
Notifications
You must be signed in to change notification settings - Fork 0
Create a new pull request by comparing changes across two branches #250
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DEPRECATED: `@angular/platform-server/testing` Use e2e tests to verify SSR behavior instead. PR Close #60915
…#60928) This will prevent usesless paramters in the docs, ex: https://angular.dev/api/core/ApplicationInitStatus#constructor This improves #60302 PR Close #60928
This commit also adds a standalone example PR Close #60930
See associated pull request for more information. PR Close #60942
`makeEnvironmentProviders` was already exported but didn't have the `@publicApi` tag. PR Close #60956
See associated pull request for more information. PR Close #60960
…le" Baseline (#60754) This effectively expands Angular's existing browser support policy to be defined as browsers covered by the "widely available" Baseline (effectively browsers released <30 months ago). Each major version will choose a date shortly before release and pin its version support to the "widely available" Baseline of that date. For now, I'm arbitrarily picking March 31st as I try to land the general infra and policy change, but we'll likely update the exact date a little closer to the v20 RC. PR Close #60754
Prepare the app for Wiz & ACX and handle unsupported features by disabling their respective UI. PR Close #60585
This commit adds the support for the `in` keyword as a relational operator, with the same precedence as the other relational operators (<,>, <=, >=) BREAKING CHANGE: 'in' in an expression now refers to the operator PR Close #58432
This commits adds the pipe usage and the impure pipe mention. PR Close #60926
Instead of stabilizing the TestBed.flushEffects() API we intend to replace it with the tick() method (equivalent of ApplicationRef.tick(). The reasoning here is that we prefer tests running the entire synchronization process (as in production apps) instead of invoking parts of the synchronization process in a way that would naver happen in a running application. PR Close #60959
Adds a flag to `parseTemplate` to enable selectorless. PR Close #60952
Adds the follow validations to the selectorless template parsing: * Local references with values are not allowed (e.g. `#foo="bar"`). * Multiple local references with the same on a component or directive are not allowed. PR Close #60952
Updates the template binder to account for the new selectorless AST nodes. This is a prerequisite to supporting template type checking of the new syntax. PR Close #60952
Introduce an alias from afterRender to afterEveryRender in preparations for the the internal rename. PR Close #60951
See associated pull request for more information. PR Close #60969
This commit moves zoneless from experimental to developer preview. * Update tag on provider API * Remove "experimental" from provider name * Move documentation from "experimental features" to "Best practives -> Performance" (at least temporarily until there is a better place) BREAKING CHANGE: `provideExperimentalZonelessChangeDetection` is renamed to `provideZonelessChangeDetection` as it is now "Developer Preview" rather than "Experimental". PR Close #60748
…`redirectTo` (#60958) Redirects in the router are handled before `canMatch` guards are evaluated. As a result, `canMatch` will not run for routes that include a redirect. Instead of silently ignoring this misconfiguration, developers should be alerted to help them understand why it doesn't behave as expected. Closes: #60957 PR Close #60958
Those weren't necessary. PR Close #61032
This intends to fix the Angular Language service issues for devtools/adev because it pulls the version from the node_modules and it currently is 14.3.0. PR Close #61128
Remove `pinDigests` as we do update docker images. PR Close #61232
This is no longer necessary since the work in the FW to rethrow in the TestBed error handler. PR Close #61243
…#61220) In the event of an invalid `schemas` field for an Angular module, an empty schema array will now be used instead of a fatal error occurring. A build will still fail in this case with the error reported as a diagnostic. However, for the language service, this allows the module to exist in the compiler registry and prevents cascading diagnostics within an IDE due to "missing" modules/components. The originating schema related errors will still be reported in the IDE. PR Close #61220
Migrate the package to using rules_js PR Close #61226
See associated pull request for more information. PR Close #61234
The template symbol builder works by finding the variables referring to template AST nodes with specific offsets and resolving them to directives. Afterwards it goes through the directives and resolves their host directives. The problem is that host directives are added with the exact same offsets as their host which means they get added once initially and again when resolving host directives. These changes resolve the issue by de-duplicating them. PR Close #61240
) Updates the language service to handle producing definition information for selectorless components and directives. PR Close #61240
This commit cleans up the embedded editor code quite a bit by making better use of current signal APIs PR Close #61242
The `waitForAsync` rule has been disabled internally which allows us to delete its code, as well as some unused adjacent code. PR Close #61260
This change uses the dev-infra preset. PR Close #61268
This package requires some work to be updated. PR Close #61294
Avoid pnpm auto-installing peer dependencies. We want to be explicit about our versions used for peer dependencies, avoiding potential mismatches. In addition, it ensures we can continue to rely on peer dependency placeholders substituted via Bazel. PR Close #61294
The code has been migrated in G3, this flag is no longer necessary. PR Close #61265
…urn type (#61312) As part of the Bazel toolchain migration we noticed that implicit types generated by the TypeScript compiler sometimes end up referencing types from other packages (i.e. cross-package imports). These imports currently work just because the Bazel `ts_library` and `ng_module` rules automatically inserted a `<amd-module name="@angular/x" />` into `.d.ts` of packages. This helped TS figure out how to import a given file. Notably this is custom logic that is not occuring in vanilla TS or Angular compilations—so we will drop this magic as part of the toolchain cleanup! To improve code quality and keep the existing behavior working, we are doing the following: - adding a lint rule that reduces the risk of such imports breaking. The failure scenario without the rule is that API goldens show unexpected diffs, and types might be duplicated in a different package! - keeping the `<amd-module` headers, but we manually insert them into the package entry-points. This should ensure we don't regress anywhere; while we also improved general safety around this above. Long-term, isolated declarations or a lint rule from eslint-typescript can make this even more robust. PR Close #61312
…the same view (#60512) We have some code that avoids `storeLet` calls for declarations only used in the same view, however we didn't previously remove the corresponding `declareLet` calls, because of the following case: ``` @let foo = something$ | async; <!-- First in the template --> {{foo}} ``` Here we need a `TNode` (created by `declareLet`) in order for DI to work correctly. Since this is only required when using pipes, we can optimize away expressions that do not have pipes. PR Close #60512
) The compiler wasn't handling `@let` declarations placed inside i18n blocks. The problem is that `assignI18nSlotDependencies` phase assigns the `target` of i18n ops much earlier than the `@let` optimization. If the `@let` ends up getting optimized because it isn't used in any child views, the pointer in the i18n instruction becomes invalid. This hadn't surfaced so far, because we didn't optimize `declareLet` ops, however once we do, we start hitting assertions that the optimized `declareLet` isn't used anywhere. These changes resolve the issue by moving the i18n phases after the `@let` optimization. PR Close #60512
Generates html for the cdk api pages based on the json files imported from angular/cdk-builds PR Close #60853
See associated pull request for more information. PR Close #61266
00c6e01
into
angular-indonesia:main
4 of 12 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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?
Other information