Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Lint on PRs instead of relying on husky #1629

Merged
merged 9 commits into from
Feb 17, 2024
Merged

Lint on PRs instead of relying on husky #1629

merged 9 commits into from
Feb 17, 2024

Conversation

mnvr
Copy link
Member

@mnvr mnvr commented Feb 17, 2024

And a few other assorted improvements.

  • Prune gitignore
  • Add type checking to the next config
  • Fix casing for Ente
  • Remove possibly unnecessary yarnrc
  • Remove global .vscode
  • Remove husky
  • Add a linting action for each PR
  • Run prettier on some leftover files
  • Prevent eslint from descending into out/

Reasoning:

That said, there is still value in doing a pre-flight check when creating a PR. If there is a non intrusive way we can think of it, this would be good to resurrect (in addition to the CI checks, not as a replacement)

@mnvr mnvr merged commit d984c60 into main Feb 17, 2024
5 checks passed
@mnvr mnvr deleted the misc-cleanup branch February 17, 2024 07:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant