Skip to content

Lint Cleanup across the project#16

Open
SamiSha99 wants to merge 9 commits into
mainfrom
lint-cleanup
Open

Lint Cleanup across the project#16
SamiSha99 wants to merge 9 commits into
mainfrom
lint-cleanup

Conversation

@SamiSha99

Copy link
Copy Markdown
Collaborator

I've ran npm run lint and fixed 59 problems, of whom are 36 errors and 23 warnings. These fix the project's lint errors without effecting how the code runs and manages data.

  • Utils.jsx renamed to Utils.js as it doesn't have any .jsx in export, then fixed all imports related to Utils.
  • EmblaCarouselArrowButtons.jsx is similar and was fixed by moving usePrevNextButtons as its own hook from its own file with the same name.
  • Removed and suppressed dead imports around the project, some left because they are suppose to be for future referecing (like importing by version).
  • Apostrophes in the project were replaced with ' to avoid the error.
  • Fixed passing children as prop instead of nested in TagButtonGroup.jsx.
  • Fixed hooks being called conditionally in GamePageDialog.jsx.

@SamiSha99
SamiSha99 requested a review from liranbr July 22, 2026 11:55
@SamiSha99 SamiSha99 self-assigned this Jul 22, 2026
@SamiSha99 SamiSha99 added the enhancement New feature or request label Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant