Skip to content

Add linting, formatting, and type checking via CI and pre-commit hooks #4

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

Merged
merged 11 commits into from
Apr 8, 2025

Conversation

steadyfall
Copy link
Owner

This PR adds a bunch of improvements to help keep the codebase clean and consistent:

  • Set up ESLint, Prettier, and TypeScript checks in a GitHub Actions workflow for automated code quality.
  • Cleaned up and centralized all linting configs — now with rules for import order and Tailwind class sorting.
  • Added Husky + lint-staged to run linting before each commit (so we catch stuff early).
  • Ran the linters across the whole project to apply the new rules.
  • Moved lint-staged config to its own file and disabled linting during production builds for better performance.

@steadyfall steadyfall merged commit c677ab1 into dev Apr 8, 2025
2 checks passed
@steadyfall steadyfall deleted the dev-set-linting-for-ci-and-precommit branch April 8, 2025 02:01
@nrpatel18
Copy link

Damn bro, CI linter and all. GEOOOOOOOOTABBBBBBB

@steadyfall steadyfall self-assigned this Apr 12, 2025
@steadyfall steadyfall added doc README updates, content tweaks, or blog formatting infra CI/CD, GitHub Actions, or deployment configs refactor internal code cleanup or restructuring (no visible change) deps Dependency or build tool updates labels Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps Dependency or build tool updates doc README updates, content tweaks, or blog formatting infra CI/CD, GitHub Actions, or deployment configs refactor internal code cleanup or restructuring (no visible change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants