entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 36 Releases Activity

misc: warn when a time related condition is used with no timeout (#69291) #1329

Merged
fpeters merged 1 commits from wip/69291-warn-on-time-related-conditions into main 2024-04-12 07:13:29 +02:00
Conversation 0 Commits 1 Files Changed 3 +31
fpeters commented 2024-03-28 13:56:42 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2024-03-28 13:56:43 +01:00
fpeters changed title from WIP: misc: warn when a time related condition is used with no timeout (#69291) to misc: warn when a time related condition is used with no timeout (#69291) 2024-03-28 14:14:50 +01:00
pmarillonnet reviewed 2024-03-28 16:26:17 +01:00
wcs/api.py Outdated
@ -1478,1 +1479,4 @@
hint['msg'] = str(e)
else:
if request.GET.get('warn-on-datetime') == 'true' and condition['type'] == 'django':
variables = re.compile(r'\b(today|now)\b')
pmarillonnet commented 2024-03-28 16:26:17 +01:00
Owner
Copy Link

Pourquoi une partie de la description du ticket (“ça pourrait commencer très bête et regarder s'il y a |date […] dans la condition”) est passée à la trappe ici ?

Pourquoi une partie de la description du ticket (“ça pourrait commencer très bête et regarder s'il y a |date […] dans la condition”) est passée à la trappe ici ?
tnoel commented 2024-04-03 13:33:33 +02:00
Owner
Copy Link

J'ajouterai qu'on pourrait aussi penser à détecter tout ce qui est « |age_in_.* » ?

J'ajouterai qu'on pourrait aussi penser à détecter tout ce qui est « |age_in_.* » ?
fpeters commented 2024-04-03 14:02:01 +02:00
Author
Owner
Copy Link

Pourquoi une partie de la description du ticket (“ça pourrait commencer très bête et regarder s'il y a |date […] dans la condition”) est passée à la trappe ici ?

Désolé j'avais raté ce commentaire, je pense que j'avais trouvé des utilisations tout à fait légitimes de |date, pour lesquelles ça me semblait mieux de ne pas inquiéter par un nouveau message.

J'ajouterai qu'on pourrait aussi penser à détecter tout ce qui est « |age_in_.* » ?

Ok j'ai du mettre une deuxième regex pour ne pas imposer le \b derrière et rester lisible.

> Pourquoi une partie de la description du ticket (“ça pourrait commencer très bête et regarder s'il y a |date […] dans la condition”) est passée à la trappe ici ? Désolé j'avais raté ce commentaire, je pense que j'avais trouvé des utilisations tout à fait légitimes de |date, pour lesquelles ça me semblait mieux de ne pas inquiéter par un nouveau message. > J'ajouterai qu'on pourrait aussi penser à détecter tout ce qui est « |age_in_.* » ? Ok j'ai du mettre une deuxième regex pour ne pas imposer le \b derrière et rester lisible.
pmarillonnet commented 2024-04-04 12:16:35 +02:00
Owner
Copy Link

C’est bon pour moi, merci.

C’est bon pour moi, merci.
fpeters force-pushed wip/69291-warn-on-time-related-conditions from b537709e5d to afa1f8879e 2024-04-03 14:01:24 +02:00 Compare
pmarillonnet approved these changes 2024-04-04 12:16:48 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ack.

Ack.
fpeters merged commit 965ed7a48c into main 2024-04-12 07:13:29 +02:00
fpeters deleted branch wip/69291-warn-on-time-related-conditions 2024-04-12 07:13:29 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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#1329
No description provided.
Delete Branch "wip/69291-warn-on-time-related-conditions"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?