Skip to content
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

Fix: Husky, Pre-commit #1376

Closed
wants to merge 2 commits into from
Closed

Conversation

vanshavenger
Copy link
Contributor

@vanshavenger vanshavenger commented Oct 2, 2024

The following PR fixes all the problems. that husky was facing and the prettier and lint errors that were conflicting on the schema .ts file

in schema.ts file earlier the wrap regex was causign lint erros as prettier remove the parens, so instead move it to New RegExp which is inbuilt and can be useful

Formatted all the files

Added more checks to ensure code standars are met
this PR also format all files to give you a brief how the formatting is done

and important to ensure all the Code standards are met and fulfilled.
Screenshot 2024-10-02 at 9 00 28 PM

Husky/pre-commit also got revamped to ensure more strict checking

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

@devsargam devsargam closed this Oct 4, 2024
@devsargam
Copy link
Collaborator

There are many file changes in your pr just had to close this since it will affect a lot of other people's prs

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