Skip to content

Add new eslint setup with todo config #410

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: chores/update-typescript-and-other-dependencies
Choose a base branch
from
Open

Add new eslint setup with todo config #410

wants to merge 3 commits into from

Conversation

casaper
Copy link
Collaborator

@casaper casaper commented Jun 19, 2022

  1. eslint installation with configuration
  2. .eslint-todo.json excludes existing code violating the new eslint rules, so no existing code has to be altered just for the linter.
  3. dev-scripts/generate-eslint-todo.ts generates .eslint-todo.json from the current code base

casaper added 3 commits June 19, 2022 18:46
@casaper
prepare tsconfig for modularity
429052a
@casaper
feat(linting): add eslint with config
152af2d 
- install required packages
- add necessary configuration
- add a exclude configuration `.eslint-todo.json` allowing to keep
  existing sources as is
@casaper
feat(lint): add developer script, generating the todo
1da8a31 
generates .eslint-todo.json from erroring existing code
@casaper casaper requested a review from tancredi as a code owner June 19, 2022 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tancredi tancredi Awaiting requested review from tancredi tancredi is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@casaper