30,909
edits
DevCamp presentation: PR to merge or not to merge: Difference between revisions
Jump to navigation
Jump to search
m
DevCamp presentation: PR to merge or not to merge (view source)
Revision as of 13:33, 24 August 2024
, 2 years ago→Quelques exemples
m (Eldy moved page PR to merge or not to merge to DevCamp presentation: PR to merge or not to merge) |
Tag: 2017 source edit |
||
| Line 34: | Line 34: | ||
Si on voit un début de transaction "''$db->begin"'' dans une boucle "for" alors on ne merge pas | Si on voit un début de transaction "''$db->begin"'' dans une boucle "for" alors on ne merge pas | ||
*il faudra sortir les transactions "begin" de la boucle ainsi que les "commit" et "rollback" | *sauf cas très particulier, il faudra sortir les transactions "begin" de la boucle ainsi que les "commit" et "rollback" | ||
'''Cas 4''' | '''Cas 4''' | ||