dorgflow icon indicating copy to clipboard operation
dorgflow copied to clipboard

new patch after a reroll patch doesn't check d.org for a new comment number

Open joachim-n opened this issue 7 years ago • 0 comments

I had this branch:

Patch for Drupal.org. Comment (expected): 2; file: 2909573-2.commerce_license.Allow-order-items-to-already-have-a-license-set-on-them-during-order-sync.patch. Automatic commit by dorgflow.
tweak
Patch for Drupal.org. Comment (expected): 2; file: 2909573-2.commerce_license.Allow-order-items-to-already-have-a-license-set-on-them-during-order-sync.patch. Automatic commit by dorgflow.
Fix

After the first patch, I spotted a problem and made one more commit -- 'tweak'. The rerolled patch had the same comment number, which is correct.

I uploaded that patch. Then I made another fix.

Patch for Drupal.org. Comment (expected): 2; file: 2909573-2.commerce_license.Allow-order-items-to-already-have-a-license-set-on-them-during-order-sync.patch. Automatic commit by dorgflow.
Fix fix
Patch for Drupal.org. Comment (expected): 2; file: 2909573-2.commerce_license.Allow-order-items-to-already-have-a-license-set-on-them-during-order-sync.patch. Automatic commit by dorgflow.
tweak
Patch for Drupal.org. Comment (expected): 2; file: 2909573-2.commerce_license.Allow-order-items-to-already-have-a-license-set-on-them-during-order-sync.patch. Automatic commit by dorgflow.
Fix

That's not correct -- the middle patch was uploaded by this point, so d.org should have been checked.

joachim-n avatar Sep 18 '17 10:09 joachim-n