3 ms·
Ability to add together.js button into the GitLab's Merge Request template. This makes the coolest tool for remote code review I've ever used.
by kirushik 12y ago
Ability to add together.js button into the GitLab's Merge Request template.
This makes the coolest tool for remote code review I've ever used.
- sytse 12y agoSounds cool! Consider contributing it back to GitLab as a project service. Make sure to mention this comment in your merge request.
- kirushik 12y agoActually it's just one <script> header, and a single (ugly) <button> tag duct-taped right into the page template. I don't think it is either upstream-ready or especially advantageous for the project. Am I wrong?
- sytse 12y agoIf you like it someone else might. And submitting it spares the you trouble of maintaining a fork when you upgrade. But making it a project service might be hard. So feel free to not submit it. Alternatively you could write a small blog post about it, maybe it inspires someone else to contribute.