if ret == QtGui.QMessageBox.Yes: return True elif ret == QtGui.QMessageBox.No: return False
Can be replaced with:
return ret == QtGui.QMessageBox.Yes
What changed between lines 29 and 30? Please make sure you're using UNIX/Linux line endings.
There's actually no need for brackets around the returned params on line 39.
Please use words_separated_by_underscores, as per PEP8. Wordsthatarechainedtogetherarehardtoreadandunderstand.
You also don't need brackets around all the things you are returning.
« Back to merge proposal
if ret == QtGui.QMessageB ox.Yes: ox.No:
return True
elif ret == QtGui.QMessageB
return False
Can be replaced with:
return ret == QtGui.QMessageB ox.Yes
What changed between lines 29 and 30? Please make sure you're using UNIX/Linux line endings.
There's actually no need for brackets around the returned params on line 39.
Please use words_separated _by_underscores , as per PEP8. Wordsthatarecha inedtogetherare hardtoreadandun derstand.
You also don't need brackets around all the things you are returning.