Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Added duration converter and time formatter #129

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

LoCrealloc
Copy link
Member

Added the duration converter and minute to time units formatter.
As those features depend on the dateutil package, it was added to the project requirements.

@LoCrealloc LoCrealloc requested review from a team and Defelo as code owners July 11, 2022 17:41
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 11, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: e964002
Status: ✅  Deploy successful!
Preview URL: https://754128c8.pydrocsid-library.pages.dev
Branch Preview URL: https://feature-durationconverter.pydrocsid-library.pages.dev

View logs

@LoCrealloc LoCrealloc self-assigned this Jul 11, 2022
@LoCrealloc LoCrealloc added enhancement New feature or request dependencies Pull requests that update a dependency file python Pull requests that update Python code labels Jul 11, 2022
PyDrocsid/translations/en.yml Outdated Show resolved Hide resolved
PyDrocsid/util.py Outdated Show resolved Hide resolved
@Tristan-H11 Tristan-H11 requested a review from a team July 12, 2022 08:17
@Defelo Defelo self-assigned this Jul 12, 2022
PyDrocsid/util.py Outdated Show resolved Hide resolved
PyDrocsid/converter.py Outdated Show resolved Hide resolved
PyDrocsid/converter.py Outdated Show resolved Hide resolved
@Tristan-H11
Copy link
Member

Die Docs sind richtig geil :)
Nach den drei requests, bin ich durch. Dann muss nur noch jmd über den Code schauen. Aber der sieht für mich eig in Ordnung aus. 🤷🏼‍♂️

@Tristan-H11
Copy link
Member

Gleiche Commits wie bspw deine Verbesserung der Docstrings oder das verbesserte Typing darfst du auch gerne squashen, wenn die eh direkt nacheinander kommen. Das macht hier nicht allzu viel aus, aber ist eigentlich eine gute Angewohnheit, solange die commits dann nicht zu groß werden. Dann hat man alles schön fachlich zusammengehörig und man reviewed nicht zwei commits am stück und beide sind inhaltlich ähnlich. ^^

Tristan-H11
Tristan-H11 previously approved these changes Jul 13, 2022
@codeclimate
Copy link

codeclimate bot commented Jul 18, 2022

Code Climate has analyzed commit e964002 and detected 0 issues on this pull request.

View more on Code Climate.

@Defelo Defelo removed their request for review October 2, 2023 10:15
@Defelo Defelo removed their assignment Oct 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file enhancement New feature or request python Pull requests that update Python code
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants