workflow: guarantee jump identifiers unicity on new workflows (#74132) #303

Merged
ecazenave merged 1 commits from wip/74132-jump-identifier-unicity into main 2023-05-22 10:14:04 +02:00
Owner

And on existing workflow with no existing identifier duplication.

And on existing workflow with no existing identifier duplication.
ecazenave changed title from WIP: workflow: guarantee jump identifiers unicity on new workflows (#74132) to workflow: guarantee jump identifiers unicity on new workflows (#74132) 2023-05-09 18:27:19 +02:00
fpeters reviewed 2023-05-12 11:32:38 +02:00
wcs/workflows.py Outdated
@ -1355,0 +1359,4 @@
identifier = getattr(item, 'identifier', None)
if identifier:
action_ids.append((item, identifier))
return action_ids
Owner

Comme ça permet de sortir d'une boucle sans passer sur toutes les valeurs, ça gagnerait à être un itérateur.

Comme ça permet de sortir d'une boucle sans passer sur toutes les valeurs, ça gagnerait à être un itérateur.
Author
Owner

Voilà.

Voilà.
wcs/workflows.py Outdated
@ -2725,0 +2739,4 @@
if value == self.identifier:
# we don't want to block if a duplication of identifier already exists
return False
for jump, jump_identifier in self.parent.parent.get_identified_jumps():
Owner

Je voulais commenter le self.parent.parent que je trouve toujours peu lisible mais il n'y avait pas vraiemnt d'autre possibilité, j'ai fait #312 comme ça il pourra être écrit self.get_workflow().get_identified_jumps() et ça sera plus clair.

Je voulais commenter le self.parent.parent que je trouve toujours peu lisible mais il n'y avait pas vraiemnt d'autre possibilité, j'ai fait https://git.entrouvert.org/entrouvert/wcs/pulls/312 comme ça il pourra être écrit self.get_workflow().get_identified_jumps() et ça sera plus clair.
Author
Owner

Voilà.

Voilà.
ecazenave added 1 commit 2023-05-15 14:46:19 +02:00
gitea/wcs/pipeline/head This commit looks good Details
125201c9e5
workflow: guarantee jump identifiers unicity on new workflows (#74132)
And on existing workflow with no existing identifier duplication.
ecazenave requested review from fpeters 2023-05-15 15:08:57 +02:00
fpeters approved these changes 2023-05-21 14:15:37 +02:00
ecazenave merged commit c894494530 into main 2023-05-22 10:14:04 +02:00
ecazenave deleted branch wip/74132-jump-identifier-unicity 2023-05-22 10:14:04 +02:00
Owner

(j'aurais attendu le prochain cycle pour merger mais ok, je vais voir pour tagguer/déployer tout de suite).

(j'aurais attendu le prochain cycle pour merger mais ok, je vais voir pour tagguer/déployer tout de suite).
Author
Owner

(j'aurais attendu le prochain cycle pour merger mais ok, je vais voir pour tagguer/déployer tout de suite).

J'ai hésité à pousser, merci pour le déploiement rapide.

> (j'aurais attendu le prochain cycle pour merger mais ok, je vais voir pour tagguer/déployer tout de suite). J'ai hésité à pousser, merci pour le déploiement rapide.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#303
No description provided.