friends-of-presta / fop_console

Prestashop Module providing a set of shell/terminal commands for developers (PrestaShop 1.7.5+)
Academic Free License v3.0
84 stars 35 forks source link

Squash commit ? #178

Open ghost opened 2 years ago

ghost commented 2 years ago

Describe the bug

When I look at the latest version, shouldn't we group the commits

When prestashop asks me to do so I complain but it's still cleaner

SebSept commented 2 years ago

you mean, squash all the commits from dev into a single one when merging dev into master ? What's the real problem ?

ghost commented 2 years ago

Je voulais dire dans nos PR essayé d'avoir moins de commits et plus compréhensible.

Pourrait devenir que Add xxx

Je parle surtout pour moi :D pour les prochaines PR du coup

SebSept commented 2 years ago

Ah, oui, oki 🙂. C'est une recommandation qui pourrait rentrer dans des recommandations de dev. Dans le fichier Contributing ?

ghost commented 2 years ago

Oui ça me semble pas mal comme recommandation.