Skip to content

Refactorización de importaciones de datetime para mantener la coherencia entre módulos #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
cuevasrja merged 18 commits into from
Dec 13, 2024
Merged

Refactorización de importaciones de datetime para mantener la coherencia entre módulos #1

cuevasrja merged 18 commits into from
Dec 13, 2024

Conversation

@Jeam-zx
Copy link

@Jeam-zx Jeam-zx commented Dec 8, 2024 •
edited
Loading

Qué?

Se ha realizado un refactor de los imports de datetime para estandarizar su uso a través de múltiples módulos y así mejorar la consistencia y legibilidad del código.

Por qué?

Utilizar un alias consistente para los imports de datetime a través de diversos módulos ayuda a mantener un código más limpio, fácil de leer y de mantener.

Cómo?

Se han modificado los siguientes archivos para utilizar un alias común (dtm) para los imports de datetime:

  • docs/auxil/tg_const_role.py
  • telegram_bot.py
  • telegram_business.py
  • telegram_chat.py
  • telegram_chatboost.py
  • telegram_chatfullinfo.py
  • telegram_chatinvitelink.py
  • telegram_chatjoinrequest.py
  • telegram_chatmember.py
  • telegram_chatmemberupdated.py
  • telegram_giveaway.py
  • telegram_message.py
  • telegram_messageonorigin.py
  • telegram_messagereactionupdated.py

Además, se han identificado otros módulos que también requieren la refactorización de las importaciones de datetime, por lo que se continuará trabajando en este esfuerzo para abarcar todos los módulos necesarios.

@Jeam-zx Jeam-zx added the enhancement New feature or request label Dec 8, 2024
@Jeam-zx Jeam-zx self-assigned this Dec 8, 2024
cuevasrja
cuevasrja approved these changes Dec 8, 2024
View reviewed changes
Copy link
Collaborator

@cuevasrja cuevasrja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Bibo-Joshi and others added 14 commits December 9, 2024 19:34
@Bibo-Joshi @cuevasrja
Full Support for Bot API 8.0 (python-telegram-bot#4566, python-telegr…
5da7633 
…am-bot#4568, python-telegram-bot#4570, python-telegram-bot#4571, python-telegram-bot#4574, python-telegram-bot#4576, python-telegram-bot#4572)
@Bibo-Joshi @Poolitzer @cuevasrja
Documentation Improvements (python-telegram-bot#4573, python-telegram…
74899e0 
…-bot#4565)

Co-authored-by: Snehashish Biswas <coderrx06@gmail.com>
Co-authored-by: poolitzer <github@poolitzer.eu>
@Bibo-Joshi @cuevasrja
Bump Version to v21.8 (python-telegram-bot#4583)
8d0f469
@dependabot @cuevasrja
Bump srvaroa/labeler from 1.11.1 to 1.12.0 (python-telegram-bot#4586)
2f75d84 
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@nemacysts @cuevasrja
Bump pylint to v3.3.2 to Improve Python 3.13 Support (python-telegr…
cf0aa2a 
…am-bot#4590)
@dependabot @cuevasrja
Bump codecov/codecov-action from 4 to 5 (python-telegram-bot#4585)
10deb18 
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@nemacysts @cuevasrja
Allow Sequence Input for allowed_updates in Application and `Up…
d216aab 
…dater` Methods (python-telegram-bot#4589)
@Bibo-Joshi @cuevasrja
Full Support for Bot API 8.1 (python-telegram-bot#4594)
4c61987
@nemacysts @cuevasrja
Use MessageLimit.DEEP_LINK_LENGTH in `helpers.create_deep_linked_ur…
95f0db1 
…l` (python-telegram-bot#4597)
@dependabot @Bibo-Joshi @cuevasrja
Bump pytest from 8.3.3 to 8.3.4 (python-telegram-bot#4596)
3869b1e 
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
@dependabot @Bibo-Joshi @cuevasrja
Update aiolimiter requirement from ~=1.1.0 to >=1.1,<1.3 (python-te…
5327695 
…legram-bot#4595)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
@Bibo-Joshi @cuevasrja
Bump Version to v21.9 (python-telegram-bot#4601)
771e687
@cuevasrja
Merge branch 'master' into refactor/unify-datetime-imports
7a44e81
@Jeam-zx
Reorder imports to unify datetime usage across modules
f15a20e
AnyaMarcanito
AnyaMarcanito approved these changes Dec 13, 2024
View reviewed changes
Copy link

@AnyaMarcanito AnyaMarcanito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@cuevasrja cuevasrja merged commit f5233a4 into master Dec 13, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cuevasrja cuevasrja cuevasrja approved these changes

@AnyaMarcanito AnyaMarcanito AnyaMarcanito approved these changes

Assignees

@Jeam-zx Jeam-zx

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Jeam-zx @cuevasrja @AnyaMarcanito @Bibo-Joshi @nemacysts