GitHubFlow PR merge idiot-proofing

Confirmation before common mistakes when merging Pull Requests in github

Poser une question, écrire un avis, ou signaler le script.

Enveloppe les lignes

// ==UserScript==
// @name         GitHubFlow PR merge idiot-proofing
// @namespace    https://gist.github.com/raveren/f08ba2673a92c582692e1a233621762f
// @version      0.3
// @author       raveren
// @description  Confirmation before common mistakes when merging Pull Requests in github
// @match        https://github.com/*/pull/*
// ==/UserScript==

(function() {
    document.querySelector('.js-merge-branch-action').addEventListener('click', function(e){
        if ( document.querySelector('#commits_tab_counter').innerHTML.trim()!='1'
            && !confirm('More than one commit, you still wanna merge?')) {
            e.preventDefault();
            e.stopPropagation();
            return false;
        }
        
        if ( document.querySelector('.text-pending') && !confirm('Build not complete yet, you still wanna merge?')) {
            e.preventDefault();
            e.stopPropagation();
            return false;
        }
    });
})();