Skip to content

Add husky#219

Merged
czerwiukk merged 7 commits intomainfrom
FCE-900
Dec 22, 2025
Merged

Add husky#219
czerwiukk merged 7 commits intomainfrom
FCE-900

Conversation

@p1003
Copy link
Contributor

@p1003 p1003 commented Dec 12, 2025

Description

Adds husky

Motivation and Context

We want pre-commit like functionality (husky does that)

@p1003 p1003 requested a review from Copilot December 12, 2025 14:18
@p1003 p1003 self-assigned this Dec 12, 2025
@linear
Copy link

linear bot commented Dec 12, 2025

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces pre-commit hooks to the repository to enforce code quality checks before commits. The implementation uses the Python-based pre-commit framework to run existing yarn scripts (lint, format, and typecheck) on relevant files.

Key Changes:

  • Added pre-commit configuration file with three local hooks for linting, formatting, and type checking
  • Updated README with installation instructions for the pre-commit framework

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.pre-commit-config.yaml Defines three local pre-commit hooks that run yarn lint, format, and typecheck commands on JavaScript/TypeScript files
README.md Adds recommendation to install the pre-commit framework

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@p1003 p1003 force-pushed the FCE-900 branch 2 times, most recently from 4091e57 to 22c8f85 Compare December 12, 2025 15:12
@p1003 p1003 requested a review from Copilot December 12, 2025 15:16
@p1003 p1003 changed the title Add pre-commit Add husky Dec 12, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@p1003 p1003 requested a review from czerwiukk December 12, 2025 15:31
@p1003 p1003 marked this pull request as ready for review December 12, 2025 15:31
@czerwiukk czerwiukk enabled auto-merge (squash) December 22, 2025 15:32
@czerwiukk czerwiukk merged commit 31dc621 into main Dec 22, 2025
2 checks passed
@czerwiukk czerwiukk deleted the FCE-900 branch December 22, 2025 15:33
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