Skip to content

refactor(ui): Organize imports and improve code style across components#195

Closed
adamshiervani wants to merge 0 commit intodevfrom
chore/upgrade-and-apply-eslint
Closed

refactor(ui): Organize imports and improve code style across components#195
adamshiervani wants to merge 0 commit intodevfrom
chore/upgrade-and-apply-eslint

Conversation

@adamshiervani
Copy link
Contributor

  • Apply eslint to all tsx and ts files.
  • Upgrade to eslint 8

Wait with merge, there are some big pending ones that needs to be merged before.

@timothystewart6
Copy link
Contributor

timothystewart6 commented Feb 26, 2025

After this is merged, we can then enforce it during CI. I've already added the task, it's just commented out in #189. However we don't want to fix the lint during CI, we just want to check it. Since you changed the behavior of npm lint to also fix, we might need another script that just checks without the fix flag.

Or you could just create a new task called lint:fix that fixes it.

@adamshiervani adamshiervani force-pushed the chore/upgrade-and-apply-eslint branch from 27905b9 to ba0c937 Compare March 25, 2025 09:23
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