Skip to content

chore: set up linting and formatting infrastructure#34

Merged
Pinkish-Warrior merged 2 commits intomainfrom
chore/setup-linting-infrastructure
Dec 15, 2025
Merged

chore: set up linting and formatting infrastructure#34
Pinkish-Warrior merged 2 commits intomainfrom
chore/setup-linting-infrastructure

Conversation

@Pinkish-Warrior
Copy link
Contributor

  • Add ESLint with TypeScript support and strict rules
  • Add Prettier for consistent code formatting (80 char, double quotes, LF)
  • Add Husky git hooks (pre-commit: lint-staged, pre-push: type-check)
  • Add lint-staged for efficient staged file checking
  • Add markdownlint for documentation consistency
  • Add EditorConfig for cross-OS/IDE consistency
  • Create comprehensive LINTS.md developer guide
  • Update package.json with lint, format, type-check, and validate scripts

This ensures code quality and consistency across the team of 4 developers working on different operating systems.

Note: Pre-push hook temporarily excludes tests until Vitest setup is complete.

- Add ESLint with TypeScript support and strict rules
- Add Prettier for consistent code formatting (80 char, double quotes, LF)
- Add Husky git hooks (pre-commit: lint-staged, pre-push: type-check)
- Add lint-staged for efficient staged file checking
- Add markdownlint for documentation consistency
- Add EditorConfig for cross-OS/IDE consistency
- Create comprehensive LINTS.md developer guide
- Update package.json with lint, format, type-check, and validate scripts

This ensures code quality and consistency across the team of 4 developers
working on different operating systems.

Note: Pre-push hook temporarily excludes tests until Vitest setup is complete.
. "$(dirname -- "$0")/_/husky.sh"

# Run lint-staged for backend
cd backend && npx lint-staged
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a big question about using Husky. I don’t think it’s necessary, and I’m worried it will actually make our work more complicated. It’s much more convenient to make intermediate commits without thinking about the linter every time, and then run one general check before pushing. So I suggest we remove .husky/pre-commit.

@Pinkish-Warrior
Copy link
Contributor Author

Pinkish-Warrior commented Dec 12, 2025 via email

@Pinkish-Warrior Pinkish-Warrior merged commit 4ae4768 into main Dec 15, 2025
1 check passed
@Pinkish-Warrior Pinkish-Warrior deleted the chore/setup-linting-infrastructure branch December 15, 2025 15:16
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

Comments