[On Hold - Dec 22] Pre commit hooks

[sidmohanty11]

• added necessary pre-commit hooks mentioned in the issue #49 under ./githooks
• added a .pre-commit-config.yaml file where we can modify any configuration of the hooks
• added a make script that shall be run to enable these hooks to run, ref: https://www.viget.com/articles/two-ways-to-share-git-hooks-with-your-team/

make enable-git-hooks

[debdutdeb]

Why not use husky? Our main repo already uses it.

https://github.com/RocketChat/Rocket.Chat/blob/develop/.husky/pre-push

[sidmohanty11]

@debdutdeb we could totally do that, I've added this according to the issue description but let me know if we want to use husky instead

[Sing-Li]

Thanks for the contrib.

Pre commit hooks are #1 deterrent to new contributors and bain of bad dev experience (turning many away from our main repo).

At this early stage of our project, we need to keep the onramp simple to attract more contributors and users.

We don't need it yet. Let's revisit this in June.

[Sing-Li]

Please focus your energy on the huge list of project contents TODOs instead of adding project infra elements.

[sidmohanty11]

Sure!