Unify datetime import in all files #4605
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* update datetime to datetime as dtm in test folder * update datetime to datetime as dtm in test/auxil, test/ext folders * fix black format errors
…cia entre módulos (#1) * Refactor datetime imports to use alias for consistency * Refactor datetime imports to use alias for consistency across modules * Refactor datetime imports to use alias for consistency in ChatMemberUpdated, Giveaway, Message, MessageOrigin modules * Refactor datetime imports to use alias for consistency in ChatJoinRequest and ChatMember modules * Full Support for Bot API 8.0 (python-telegram-bot#4566, python-telegram-bot#4568, python-telegram-bot#4570, python-telegram-bot#4571, python-telegram-bot#4574, python-telegram-bot#4576, python-telegram-bot#4572) * Documentation Improvements (python-telegram-bot#4573, python-telegram-bot#4565) Co-authored-by: Snehashish Biswas <[email protected]> Co-authored-by: poolitzer <[email protected]> * Bump Version to v21.8 (python-telegram-bot#4583) * Bump `srvaroa/labeler` from 1.11.1 to 1.12.0 (python-telegram-bot#4586) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump `pylint` to v3.3.2 to Improve Python 3.13 Support (python-telegram-bot#4590) * Bump `codecov/codecov-action` from 4 to 5 (python-telegram-bot#4585) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Allow `Sequence` Input for `allowed_updates` in `Application` and `Updater` Methods (python-telegram-bot#4589) * Full Support for Bot API 8.1 (python-telegram-bot#4594) * Use `MessageLimit.DEEP_LINK_LENGTH` in `helpers.create_deep_linked_url` (python-telegram-bot#4597) * Bump `pytest` from 8.3.3 to 8.3.4 (python-telegram-bot#4596) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Bibo-Joshi <[email protected]> * Update `aiolimiter` requirement from ~=1.1.0 to >=1.1,<1.3 (python-telegram-bot#4595) Signed-off-by: dependabot[bot] <[email protected]> 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 <[email protected]> * Bump Version to v21.9 (python-telegram-bot#4601) * Reorder imports to unify `datetime` usage across modules --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Bibo-Joshi <[email protected]> Co-authored-by: Snehashish Biswas <[email protected]> Co-authored-by: poolitzer <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Luis Pérez <[email protected]> Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com> Co-authored-by: Juan Andrés Cuevas <[email protected]>
…tion_error_date to use dtm.datetime
harshil21
reviewed
Dec 13, 2024
Bibo-Joshi
requested changes
Dec 13, 2024
…sinessmessagesdeletedhandler.py and test_jobqueue.py
Bibo-Joshi
requested changes
Dec 14, 2024
There was a problem hiding this comment.
Thanks for the update! Now unfortunately something went wrong with merging master: Apparently you reverted the merge in dda1e7c - please revert the revert 😅
This reverts commit dda1e7c.
Bibo-Joshi
approved these changes
Dec 15, 2024
|
Thank you very much for the contribution :) |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Unifiy datetime import in all files
@Jeam-zx
@Migueldsc12
@AnyaMarcanito
@henryg311
@cuevasrja
Summary
Convert all datetime imports of the project to
import datetime as dtm. Since there was no standard in importing functions from this library.Tests
It was verified that all changes passed the static check performed by the linters and other project tools with the
pre-commit run -acommand.Check-list
This checklist is a non-exhaustive reminder of things that should be done before a PR is merged, both for you as contributor and for the maintainers. Feel free to copy (parts of) the checklist to the PR description to remind you or the maintainers of open points or if you have questions on anything.
Issue
Resolves #4577