Skip to content

[pull] master from angular:master#377

Merged
pull[bot] merged 7 commits intoPandinosaurus:masterfrom
angular:master
Oct 2, 2019
Merged

[pull] master from angular:master#377
pull[bot] merged 7 commits intoPandinosaurus:masterfrom
angular:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Oct 2, 2019

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

StephenFluin and others added 7 commits October 2, 2019 13:20
…mode (#32955)

Similarly to `ts_library` compilation actions, the `ng_module` compile action should include
the current compile mode in the action description. This makes it consistent with `ts_library`
targets and also avoids confusion when both output flavors are requested.

Currently when both output flavors are requested (e.g. in the `ng_package` rule), both
devmode and prodmode compilations have the same action name. This is confusing and
looks like the given target is built *twice* due to a bug (which is obviously not the case though)

PR Close #32955
Angular uses tslint:recommended by default. The default for no-angle-bracket-type-assertion is true
See https://github.com/palantir/tslint/blob/master/src/configs/recommended.ts#L69

PR Close #31042
Adds information about the NgModule a Directive is declared in when the
Directive class name is hovered over, in the form

```
(directive) NgModule.Directive: class
```

Closes #32565

PR Close #32763
@pull pull bot added the ⤵️ pull label Oct 2, 2019
@pull pull bot merged commit e7729b6 into Pandinosaurus:master Oct 2, 2019
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.

6 participants