Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-54119

ESlint rules are not applied on the client

    Details

    • Sprint:
      Evergreen - Milestone 2
    • Similar Issues:

      Description

      Problem statement

      Since the typescript migration on the evergree-client, we still apply ESLint, but it's a no-op. (I suppose it's looking for .js, and there's none anymore).

      Expected behavior

      ESLint should either be ditched on the client side, or enabled through some intermediate tooling (there are apparently a few initiatives around this like https://github.com/eslint/typescript-eslint-parser or even https://palantir.github.io/tslint/)

      I think this issue is more than trivially important since ESLint, and make lint for our case, gives a false sense of consistency preservation, when actually now ESLint is basically ignored for the client side.

        Attachments

          Activity

          batmat Baptiste Mathus created issue -
          batmat Baptiste Mathus made changes -
          Field Original Value New Value
          Assignee R. Tyler Croy [ rtyler ]
          batmat Baptiste Mathus made changes -
          Description h3. Problem statement

          Since the typescript migration on the evergree-client, we still apply ESLint, but it's a no-op. (I suppose it's looking for .js, and there's none anymore).

          h3. Expected behavior

          ESLint should either be ditched on the client side, or enabled through some intermediate tooling (there are apparently a few initiatives around this like https://github.com/eslint/typescript-eslint-parser or even https://palantir.github.io/tslint/)
          h3. Problem statement

          Since the typescript migration on the evergree-client, we still apply ESLint, but it's a no-op. (I suppose it's looking for .js, and there's none anymore).

          h3. Expected behavior

          ESLint should either be ditched on the client side, or enabled through some intermediate tooling (there are apparently a few initiatives around this like https://github.com/eslint/typescript-eslint-parser or even https://palantir.github.io/tslint/)

          I think this issue is more than trivially important since ESLint, and {{make lint}} for our case, gives a false sense of consistency preservation, when actually now ESLint is basically ignored for the client side.
          rtyler R. Tyler Croy made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          rtyler R. Tyler Croy made changes -
          Assignee R. Tyler Croy [ rtyler ]

            People

            • Assignee:
              rtyler R. Tyler Croy
              Reporter:
              batmat Baptiste Mathus
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: