Promote "cannot find Directive/Pipe" warning to error #6519

Activity

1. kegluneq

   added

   comp: dart-transformer

   action: discuss

   effort1: hours

   on Jan 15, 2016
2. kegluneq commented on Jan 15, 2016

   

   kegluneq

   on Jan 15, 2016

   Author

   More actions

   Any objections to this? The result would be broken builds if/when @Injectable or an import is omitted for a type which appears in the directives parameter of a View or Component.
3. kegluneq commented on Jan 15, 2016

   

   kegluneq

   on Jan 15, 2016

   Author

   More actions

   @yjbanov @jbdeboer @tbosch @vsavkin
4. jbdeboer commented on Jan 15, 2016

   

   jbdeboer

   on Jan 15, 2016

   More actions

   I love errors and fully endorse your promotion.
5. tbosch commented on Jan 16, 2016

   

   tbosch

   on Jan 16, 2016

   Contributor

   More actions

   +1
6. kegluneq

   self-assigned this

   on Jan 19, 2016
7. kegluneq

   removed

   action: discuss

   on Jan 19, 2016
8. kegluneq

   added a commit that references this issue on Jan 19, 2016

   feat(dart/transform): Promote missing Directive warning to error

   ...

   This issue will close once commit 1aad0e0 is merged into the 'master' branch.

   1aad0e0
9. mhevery

   added

   P1: urgent

   type: bug/fix

   on Jan 20, 2016
10. mhevery

    closed this as completedin 47a3b4don Jan 22, 2016
11. angular-automatic-lock-bot commented on Sep 7, 2019

    

    angular-automatic-lock-botBot

    on Sep 7, 2019

    More actions

    This issue has been automatically locked due to inactivity.
    Please file a new issue if you are encountering a similar or related problem.

    Read more about our automatic conversation locking policy.

    _{This action has been performed automatically by a bot.}
12. angular-automatic-lock-bot

    locked and limited conversation to collaborators on Sep 7, 2019