Skip to content

Internal lint issues on Sheriff repo #172

@tomwhite007

Description

@tomwhite007

This is only a suggestion—I'm happy for you to close it if you want. I'm just looking to improve DX.

Expected: As a developer, I would like to see only lint issues affecting my current file.

Actual: For .json and .html files in this repo, I see lint errors that pertain to .ts files. e.g.

Image

.js files also get lint errors for using CommonJS (require(...)) imports.

I propose you let me submit a small PR to convert eslint.config.js in the root to the format of a freshly created Nx repo. This would separate .ts lint rules away from .js, .json and .html files. If you have any special requirements that aren't standard linting rules, you could mention them here.

I'm suggesting this to improve my DX as I work on #165

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions