Skip to content

[pull] master from angular:master#540

Merged
pull[bot] merged 25 commits intoashu8912:masterfrom
angular:master
Aug 17, 2021
Merged

[pull] master from angular:master#540
pull[bot] merged 25 commits intoashu8912:masterfrom
angular:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Aug 16, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

dario-piotrowicz and others added 12 commits August 16, 2021 09:23
fix typo put in the ExtractorHosts's resourceNameToFileName comment
in which "later on" is spelled "lateron"

PR Close #43160
…omment (#43147)

remove the extra 'diagnostic' present in the comment for the
ResourceHost's readResource method

PR Close #43147
…ations (#43102)

The management of `browserUrlTree` currently has several problems with
correctly tracking the actual state of the browser.

This change makes the Router eagerly update the `browserUrlTree` when
handling navigations triggered by browser events (i.e., not 'imperative'). This
is because with those types of navigations, the browser URL bar is
_already_ updated. If we do not update the internal tracking of the
`browserUrlTree`, we will be out of sync with the real URL if the
navigation is rejected.

It would be best if we could remove `browserUrlTree` completely, but doing that
would require a lot more investigation and is blocked by #27059 because
the SpyLocation used in tests does not emulate real browser behavior.

fixes #43101

PR Close #43102
…dev-infra-private-builds (#43061)

Remove the dev-infra directory and complete the migration to using the code generated by the
angular/dev-infra repository.

PR Close #43061
Update the ng-dev config for Github to include the main branch name.

PR Close #43061
…erformances and coding standards (#42998)

HttpParams uses custom encoding function "standardEncoding" to encode query string preserving specific charachters. This refactoring aims to improve performances and code quality of that function by using a RegExp and a Map object instead of a chain of replace functions.

PR Close #42998
Update the prefix used for the circle cache key to use new cache base.

PR Close #43177
@pull pull bot added the ⤵️ pull label Aug 16, 2021
alan-agius4 and others added 13 commits August 16, 2021 11:37
Update renovate config to allow GitHub workflow updates

PR Close #43146
Update to the latest @angular/dev-infra as it includes the fix to no longer include
husky installs in the published package.json

PR Close #43179
This import is not used in the file, so can be removed.

PR Close #43129
…ts (#43129)

The compliance tests can check source-map segments against expectations
encoded into the expectation files. Previously, the encoding of the expected
segment was only delimited by whitespace, but this made it difficult to identify
segments that started or ended with whitespace.

Now these segment expectations are wrapped in double-quotes which makes
it easier to read and understand the expectation files.

PR Close #43129
This commit removes 9 cycles in the dependency graph of the compiler code.

PR Close #43129
This function is general purpose and by moving it into the
`chars.ts` file along with similar helpers, it can be reused
in the lexer, for instance.

PR Close #43129
This is a simple tidy up commit to move to the more specific `===`
comparison operator in the HTML lexer/parser.

PR Close #43129
… interpolations (#43129)

Such interpolations turned up during internal testing at Google, so this
commit adds a test to prevent regressions.

PR Close #43129
…butes (#43129)

This tests a scenario that was failing in an internal project.

PR Close #43129
…w.md` (#43180)

The previously used code (`<live-example stackblitz="router">`) would
try to target the `router.stackblitz.json` file inside the
(non-existent) `routing-overview` example (same as the guide's name).

This commit fixes the code to correctly express the original intention
of targeting the default `stackblitz.json` file of the `router` example
with `<live-example name="router">`.

Fixes #43167

PR Close #43180
…scProgram (#43148)

remove the NgtscProgram's private field closureCompilerEnabled as that
is not being used in the class itself

PR Close #43148
Class static properties with initializers that cause code execution (for example, call expressions or new expressions) have the potential to cause side effects at module evaluation. This is similar in effect to module level code. As a result, optimizers can not safely remove a class with such a static property as the potential side effects may have meaningful effects on the state of the application execution. To allow classes with these type of static properties to be optimized and removed if unused, the initializer expressions for the static properties can be annotated as pure. This annotation provides a signal to an optimizer that the expression does not have any potential side effects and is useful in cases where static analysis can not currently prove that there are, in fact, no side effects caused by the initializer.

PR Close #43064
…#43064)

Class static properties with initializers that cause code execution (for example, call expressions or new expressions) have the potential to cause side effects at module evaluation. This is similar in effect to module level code. As a result, optimizers can not safely remove a class with such a static property as the potential side effects may have meaningful effects on the state of the application execution. To allow classes with these type of static properties to be optimized and removed if unused, the initializer expressions for the static properties can be annotated as pure. This annotation provides a signal to an optimizer that the expression does not have any potential side effects and is useful in cases where static analysis can not currently prove that there are, in fact, no side effects caused by the initializer.

PR Close #43064
@pull pull bot merged commit 3cb1f18 into ashu8912:master Aug 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.