fix(@angular-devkit/build-angular): fix duplicate lint messages

[alan-agius4]

Closes #11633

[filipesilva]

Just blocking it until a repro is shown in the original issue.

[filipesilva]

I think you can repro if you update the lint target of the test project to this:

        "lint": {
          "builder": "@angular-devkit/build-angular:tslint",
          "options": {
            "tsConfig": [
              "src/tsconfig.app.json",
              "src/tsconfig.spec.json"
            ],
            "exclude": [
              "**/node_modules/**"
            ]
          }
        },

To fix the problem you'd need to dedupe the result array before reading it around

angular-cli/packages/angular_devkit/build_angular/src/tslint/index.ts

Line 96 in d7e7cd2

[alan-agius4]

thanks for your help I managed to replicate the issue

…

On Tue, 24 Jul 2018 at 11:42, Filipe Silva ***@***.***> wrote: I think you can repro if you update the lint target of the test project to this: "lint": { "builder": ***@***.***/build-angular:tslint", "options": { "tsConfig": [ "src/tsconfig.app.json", "src/tsconfig.spec.json" ], "exclude": [ "**/node_modules/**" ] } }, To fix the problem you'd need to dedupe the result array before reading it around https://github.com/angular/angular-cli/blob/d7e7cd22824e4b6349c0071326f3c0090265b469/packages/angular_devkit/build_angular/src/tslint/index.ts#L96 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#11636 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AQv-WmMxynCf4iHqiHhyxnNCgLoU6V0bks5uJuwegaJpZM4VcGzL> .

[alan-agius4]

This is due that duplicate messages are within the same entry since under the hood we are using tslint formatter to format the messages with will convert and array of results to a string

[filipesilva]

LGTM, thanks for taking this on!

[angular-automatic-lock-bot]

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.}