Sitelet https://github.com/angular/angular/compare/10.2.2...10.2.3
Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: angular/angular
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 10.2.2
Choose a base ref
...
head repository: angular/angular
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 10.2.3
Choose a head ref
  • 11 commits
  • 24 files changed
  • 6 contributors

Commits on Nov 4, 2020

  1. Configuration menu
    Copy the full SHA
    08e077c View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2020

  1. fix(compiler): ensure that i18n message-parts have the correct source…

    …-span (#39589)
    
    In an i18n message, two placeholders next to each other must have
    an "empty" message-part to separate them. Previously, the source-span
    for this message-part was pointing to the wrong original location.
    This caused problems in the generated source-maps and lead to extracted
    i18n messages from being rendered incorrectly.
    
    PR Close #39589
    petebacondarwin authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    e67a331 View commit details
    Browse the repository at this point in the history
  2. refactor(compiler): store the fullStart location on `ParseSourceSpa…

    …n`s (#39589)
    
    The lexer is able to skip leading trivia in the `start` location of tokens.
    This makes the source-span more friendly since things like elements
    appear to begin at the start of the opening tag, rather than at the
    start of any leading whitespace, which could include newlines.
    
    But some tooling requires the full source-span to be available, such
    as when tokenizing a text span into an Angular expression.
    
    This commit simply adds the `fullStart` location to the `ParseSourceSpan`
    class, and ensures that places where such spans are cloned, this
    property flows through too.
    
    PR Close #39589
    petebacondarwin authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    1aee8b3 View commit details
    Browse the repository at this point in the history
  3. refactor(compiler): capture fullStart locations when tokenizing (#3…

    …9589)
    
    This commit ensures that when leading whitespace is skipped by
    the tokenizer, the original start location (before skipping) is captured
    in the `fullStart` property of the token's source-span.
    
    PR Close #39589
    petebacondarwin authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    ff31b43 View commit details
    Browse the repository at this point in the history
  4. fix(compiler): skipping leading whitespace should not break placehold…

    …er source-spans (#39589)
    
    Tokenized text node may have leading whitespace skipped from their
    source-span. But the source-span is used to compute where there are
    interpolated blocks, resulting in placeholder nodes whose source-spans
    are offset by the amount of skipped characters.
    
    This fix uses the `fullStart` location of text source-spans for computing
    the source-span of placeholders, so that they are accurate.
    
    Fixes #39195
    
    PR Close #39589
    petebacondarwin authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    2b684b7 View commit details
    Browse the repository at this point in the history
  5. docs: typo fix for 'Intall' (#39585)

    'Intall' should be 'Install'
    PR Close #39585
    damingerdai authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    7bd0133 View commit details
    Browse the repository at this point in the history
  6. fix(compiler-cli): do not drop non-Angular decorators when downleveli…

    …ng (#39577)
    
    There is a compiler transform that downlevels Angular class decorators
    to static properties so that metadata is available for JIT compilation.
    The transform was supposed to ignore non-Angular decorators but it was
    actually completely dropping decorators that did not conform to a very
    specific syntactic shape (i.e. the decorator was a simple identifier, or
    a namespaced identifier).
    
    This commit ensures that all non-Angular decorators are kepts as-is
    even if they are built using a syntax that the Angular compiler does not
    understand.
    
    Fixes #39574
    
    PR Close #39577
    petebacondarwin authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    1c6cf8a View commit details
    Browse the repository at this point in the history
  7. fix(compiler-cli): avoid duplicate diagnostics about unknown pipes (#…

    …39517)
    
    TCB generation occasionally transforms binding expressions twice, which can
    result in a `BindingPipe` operation being `resolve()`'d multiple times. When
    the pipe does not exist, this caused multiple OOB diagnostics to be recorded
    about the missing pipe.
    
    This commit fixes the problem by making the OOB recorder track which pipe
    expressions have had diagnostics produced already, and only producing them
    once per expression.
    
    PR Close #39517
    alxhub authored and mhevery committed Nov 6, 2020
    Configuration menu
    Copy the full SHA
    861e4fa View commit details
    Browse the repository at this point in the history

Commits on Nov 9, 2020

  1. build(docs-infra): print the git commit when deploying to Firebase (#…

    …39596)
    
    The commit updates the AIO deployment script to also print the commit
    SHA. This makes it easier to check whether a version has been
    successfully deployed, by comparing the commit SHA from the CI job with
    the SHA in the version string in the footer of the AIO app.
    
    PR Close #39596
    gkalpak authored and mhevery committed Nov 9, 2020
    Configuration menu
    Copy the full SHA
    1357d84 View commit details
    Browse the repository at this point in the history
  2. ci: log commands output when deploying angular.io to Firebase (#39596)

    In #39470, the `deploy-to-firebase.sh` script (used to deploy AIO to
    Firebase when building an upstream branch), was replaced by an
    equivalent JS script. In this new `deploy-to-firebase.js` script, we
    were overly aggressive with suppressing command output, which made it
    hard to investigate failures ([example failing CI job][1]).
    
    This commit updates the `deploy-to-firebase.js` script to capture
    command output as usual in the CI job logs. This makes the output
    similar to the one generated by the old [deploy-to-firebase.sh][2]
    script ([example CI logs][3]).
    
    One concern with capturing command output is having the value of a
    secret environment variables leaked in the logs. This is not the case
    here, since:
    1. The secret env vars are not printed from the commands that use them.
    2. CircleCI will [mask the values of secret env vars][4] in the output.
    
    As an extra precaution (although not strictly necessary), we run `yarn`
    with the `--silent` option, which avoid echoing the executed yarn
    commands.
    
    [1]: https://circleci.com/gh/angular/angular/849310
    [2]: https://github.com/angular/angular/blob/3b0b7d22109c79b4dceb/aio/scripts/deploy-to-firebase.sh
    [3]: https://circleci.com/gh/angular/angular/848109
    [4]: https://circleci.com/docs/2.0/env-vars/#secrets-masking
    
    PR Close #39596
    gkalpak authored and mhevery committed Nov 9, 2020
    Configuration menu
    Copy the full SHA
    ae71b76 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    0786c59 View commit details
    Browse the repository at this point in the history
Loading