feat: add commitlint

[netanelC]

Adds to the repo commitlint so the commits will fit our standards.
In addition, I added workflow that validates that the PR title also fits the standards

[shimoncohen]

Please check this out for PR title as well:
https://github.com/marketplace/actions/pull-request-linter-action
If i'm not mistaken, the current workflow doesn't check the PR title (the action won't trigger when it changes anyway).

[shimoncohen]

Please move file to configs folder as done here:
https://github.com/MapColonies/shared-workflows/pull/28/files

[netanelC]

I was not finished 😢

[shimoncohen]

Sorry about the early review 😋 I will wait.

[netanelC]

The commit config should sit on the repo itself like this

[shimoncohen]

Well, you removed the workflow on this commit - a5434cb.
This totally changes the PR meaning from adding the workflow to simply adding commitlint.
Why not just add the workflows and enforce them for every PR here? Because I don't want us to add files like package.lock to this repo, it doesn't really belong here IMO.

[netanelC]

I thought so too, but it is meaningless to enforce them in a workflow. That way, the commits are already pushed and the checks will just fail..
I want to prevent the wrong commit message from the beginning. Can you think of an alternative besides husky?

And about the pr-title workflow, it is not belong here so I opened a new PR #30

[netanelC]

Won't do. As discussed, it is not good to put typescript in workflows repo

[shimoncohen]

Won't do. As discussed, it is not good to put typescript in workflows repo

It's important to note that we said this isn't good in the current state of the repo. We still need to add the workflows for this.