Skip to content
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

Resolve eslint max-depth warnings #1249

Open
samayer12 opened this issue Oct 9, 2024 · 1 comment · May be fixed by #1255
Open

Resolve eslint max-depth warnings #1249

samayer12 opened this issue Oct 9, 2024 · 1 comment · May be fixed by #1255
Labels
tech-debt

Comments

@samayer12
Copy link
Collaborator

samayer12 commented Oct 9, 2024 •
edited
Loading

Describe what should be investigated or refactored

ESlint has several warnings for max-depth violations across the codebase. Resolve these warnings.

Definition of Done

  • There are zero eslint warnings for max-depth
  • max-depth is set to error from warn in .eslintrc.json

Additional context

Run the linter with npm format:check
@samayer12 samayer12 added tech-debt good first issue Good for newcomers labels Oct 9, 2024
@tamirazrab
Copy link

Hi, I'd like to work on this.

@tamirazrab tamirazrab linked a pull request Oct 10, 2024 that will close this issue
Open
5 tasks
@cmwylie19 cmwylie19 added this to the v0.39.0 milestone Oct 23, 2024
@cmwylie19 cmwylie19 removed the good first issue Good for newcomers label Oct 28, 2024
@cmwylie19 cmwylie19 removed this from the v0.39.0 milestone Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech-debt
Projects
Pepr Project Board
Status: 🆕 New
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: eslint max depth warnings tamirazrab/pepr
3 participants
@cmwylie19 @samayer12 @tamirazrab