Skip to content

Additional ESLint cleanup#499

Merged
anselmbradford merged 4 commits intomainfrom
ans_linter_cleanup
Feb 19, 2026
Merged

Additional ESLint cleanup#499
anselmbradford merged 4 commits intomainfrom
ans_linter_cleanup

Conversation

@anselmbradford
Copy link
Copy Markdown
Member

Changes

  • Fix some linting errors.
  • Ignore config files in ESLint.
  • Turn off react/prop-types rule.
  • Add node types.

How to test this PR

  1. yarn lint should have less errors than main.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 18, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-02-19 21:06 UTC

@anselmbradford anselmbradford enabled auto-merge (squash) February 19, 2026 17:22
Copy link
Copy Markdown
Collaborator

@flacoman91 flacoman91 left a comment

Choose a reason for hiding this comment

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

This works. we definitely need to get to cleaning up the errors

Image

@anselmbradford anselmbradford merged commit 87b7024 into main Feb 19, 2026
3 checks passed
@anselmbradford anselmbradford deleted the ans_linter_cleanup branch February 19, 2026 21:06
@anselmbradford
Copy link
Copy Markdown
Member Author

This works. we definitely need to get to cleaning up the errors

In your screenshot, does that just need to be updated to {}?

@flacoman91
Copy link
Copy Markdown
Collaborator

This works. we definitely need to get to cleaning up the errors

In your screenshot, does that just need to be updated to {}?

I just saw it when I ran yarn lint

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