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

Chore/es lint #26

Merged
merged 25 commits into from
Nov 2, 2023
Merged

Chore/es lint #26

merged 25 commits into from
Nov 2, 2023

Conversation

JohanHjelsethStorstad
Copy link
Member

@JohanHjelsethStorstad JohanHjelsethStorstad commented Oct 31, 2023

Adds a lot of es-lint rules

  • Mostly copied from veven
  • Removed some unnecessary things like where one can use the var keyword. We have no-var anyways.
  • One major difference: no (unnecessary) semicolons
  • Also added linting specific to typescript

I have also done the linting of this project using these new rules
The new linter also caught an error in the augmentation of next-auth that gave no types for one variable. This has been changed, but will probably be changed soon anyways

@JohanHjelsethStorstad JohanHjelsethStorstad merged commit 05ba641 into main Nov 2, 2023
2 checks passed
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.

1 participant