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

misc: use a single word for each time unit (#88822) #1330

Merged
fpeters merged 1 commits from wip/88822-single-timewords into main 2024-03-29 08:34:32 +01:00
Conversation 0 Commits 1 Files Changed 2 +18 -16
fpeters commented 2024-03-28 14:14:42 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2024-03-28 14:14:42 +01:00
gitea/wcs/pipeline/head This commit looks good Details
261bbf04c1 misc: use a single word for each time unit (#88822)
fpeters changed title from WIP: misc: use a single word for each time unit (#88822) to misc: use a single word for each time unit (#88822) 2024-03-28 14:23:06 +01:00
fpeters reviewed 2024-03-28 14:25:44 +01:00
wcs/qommon/humantime.py Outdated
@ -43,0 +39,4 @@
((_('minute'), _('minutes'), _('minute(s)')), _minute),
((_('second'), _('seconds'), _('second(s)')), 1),
((_('month'), _('months'), _('month(s)')), _month),
((_('year'), _('years'), _('year(s)')), _year),
fpeters commented 2024-03-28 14:25:44 +01:00
Author
Owner
Copy Link

En passant je modifie un peu l'ordre des unités, pour faire moins rollercoaster.

jours ↘ heures ↗mois ↗années ↘minutes ↘ secondes
→ jours ↘ heures ↘ minutes ↘ secondes ↗mois ↗années

En passant je modifie un peu l'ordre des unités, pour faire moins rollercoaster. jours ↘ heures ↗mois ↗années ↘minutes ↘ secondes → jours ↘ heures ↘ minutes ↘ secondes ↗mois ↗années
pmarillonnet approved these changes 2024-03-28 16:12:25 +01:00
pmarillonnet left a comment
Owner
Copy Link

Juste un petit truc à redire, à prendre ou pas. Ack.

Juste un petit truc à redire, à prendre ou pas. Ack.
wcs/qommon/humantime.py Outdated
@ -63,2 +58,2 @@
seconds = seconds + int(m.group(1)) * quantity
break
for (word1, word2, dummy), quantity in _humandurations:
m = re.search(r'(\d+)\s*\b(%s|%s)\b' % (word1, word2), humanduration)
pmarillonnet commented 2024-03-28 16:12:04 +01:00
Owner
Copy Link

La PR venant complexifier un peu cette expression régulière, j’aurais bien vu une petite ligne de commentaire juste au dessus, pour expliquer que l’unité peut être exprimée au singulier ou au pluriel, les deux étant d’intérêt ici.

La PR venant complexifier un peu cette expression régulière, j’aurais bien vu une petite ligne de commentaire juste au dessus, pour expliquer que l’unité peut être exprimée au singulier ou au pluriel, les deux étant d’intérêt ici.
fpeters commented 2024-03-29 08:34:27 +01:00
Author
Owner
Copy Link

En effet, j'ai ajouté un petit commentaire.

En effet, j'ai ajouté un petit commentaire.
fpeters force-pushed wip/88822-single-timewords from 261bbf04c1 to c8ffe45311 2024-03-28 21:44:49 +01:00 Compare
fpeters merged commit 81f2abeab2 into main 2024-03-29 08:34:32 +01:00
fpeters deleted branch wip/88822-single-timewords 2024-03-29 08:34:32 +01: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
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#1330
No description provided.
Delete Branch "wip/88822-single-timewords"

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?