This addition needs to come with some specs.
You should also squash all this commits to one to avoid polluting the project's history.

This addition needs to come with some specs.

As in? The summary and source comments are self explaining...

You should also squash all this commits to one to avoid polluting the project's history.

I understand but do not have the option on the web interface. Those changes have been done on the web not a local client ( note: i do not use Git a lot ) .

I strongly suggest you learn to use git, it'll make it much much easier for everyone reviewing the PR, and it's a valuable skill for yourself.

By specs, @j8r means adding automated tests in the specs directory.

@Wulfklaue , the basics of git aren't that hard, only if you use more advanced features.
Basically with git pull https://github.com/Wulfklaue/crystal, git reset --soft HEAD^(to remove commits), git commit -am "my message" and git push -f(but try avoiding -f when possible) you can do the job.
The best will be to use git rebase -i, but this is advanced.
For best practises, you can browse the project's history (PR, commits) to see how people do, and learn from external docs.

This Is included in #5627.