Skip to content

Add husky precommit check in the repo.#171

Merged
Tatsinnit merged 3 commits intoAzure:mainfrom
Tatsinnit:feat/enable-husky
Jul 2, 2025
Merged

Add husky precommit check in the repo.#171
Tatsinnit merged 3 commits intoAzure:mainfrom
Tatsinnit:feat/enable-husky

Conversation

@Tatsinnit
Copy link
Copy Markdown
Member

@Tatsinnit Tatsinnit commented Jun 24, 2025

This will add pre-commit check for the individual user making the prettier bug go un-noticed. The screenshot will tell you the rest of the story. (Also: this could complete replace #170 ) ❤️

Latest

Screenshot 2025-06-28 at 10 54 29 AM

Before Fix:

Screenshot 2025-06-25 at 10 45 59 AM

After Fix

Screenshot 2025-06-25 at 10 47 09 AM

Thanks

Signed-off-by: Tatsat Mishra <tamishra@microsoft.com>
@Tatsinnit Tatsinnit requested a review from Copilot June 24, 2025 22:50
@Tatsinnit Tatsinnit self-assigned this Jun 24, 2025
@Tatsinnit Tatsinnit added the enhancement New feature or request label Jun 24, 2025
Copy link
Copy Markdown

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 adds a husky pre-commit check to the repository to ensure that tests pass and code formatting is validated before each commit.

  • Updates package.json to include husky as a dependency and a "prepare" script
  • Introduces a new pre-commit hook in the .husky directory that runs tests and checks formatting

Reviewed Changes

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

File Description
package.json Added husky dependency and script configuration for pre-commit setup
.husky/pre-commit Created a pre-commit hook that runs tests and format-check commands

bosesuneha
bosesuneha previously approved these changes Jun 25, 2025
Copy link
Copy Markdown
Member

@bosesuneha bosesuneha left a comment

Choose a reason for hiding this comment

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

lgtm

Signed-off-by: Tatsat Mishra <tamishra@microsoft.com>
bosesuneha
bosesuneha previously approved these changes Jun 27, 2025
@Tatsinnit Tatsinnit merged commit 00ca972 into Azure:main Jul 2, 2025
6 checks passed
@davidgamero davidgamero mentioned this pull request Mar 25, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants