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.

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

Address ESLint errors from recommended plugins #2441

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

dotNomad
Copy link
Collaborator

This PR is a follow-up to #2440 that addresses all of the new rules that are failing.

Intent

Resolves #1293

Type of Change

    • Bug Fix
    • New Feature
    • Breaking Change
    • Documentation
    • Refactor
    • Tooling

Approach

Rather than change every any I figured we could make a follow-up issue to remove the // eslint-disable-next-line @typescript-eslint/no-explicit-any added here since those are a bit complex.

The work I did do here that wasn't auto-fixed was removed the async promise executors. To see why this is recommended I suggest reading the no-async-promise-executor rule docs: https://eslint.org/docs/latest/rules/no-async-promise-executor

Directions for Reviewers

Test the changed code in the following flows:

  • New Deployment flows
  • Selecting a New or Existing Config
  • The home view's handleFileInitiatedDeploymentSelection which fires when the entrypoint button is pressed

Most everything else is disabling any rules and changing to const where applicable.

Copy link
Collaborator

@marcosnav marcosnav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! It catched a lot of things!

@dotNomad
Copy link
Collaborator Author

Nice! It catched a lot of things!

Yeah I'm really happy to get these recommended rules in to catch things like this!

@dotNomad dotNomad merged commit e90edcd into dotnomad/up-eslint Nov 18, 2024
14 checks passed
@dotNomad dotNomad deleted the dotnomad/eslint-9-errs branch November 18, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants