-
-
Notifications
You must be signed in to change notification settings - Fork 20
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.
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
feat: integrate pyromod patches #1
Conversation
hydrogram/client.py
Outdated
inline_message_id: Optional[Union[str, List[str]]] = None, | ||
): | ||
""" | ||
Creates a listener and waits for it to be fulfilled. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Documentation should be rewritten to follow Hydrogram's style.
4d447b0
to
5752af6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good, but the docstrings should follow the Hydrogram format so as not to have dissonance in the docs, and we should also add the pyromod functions to the documentation with examples.
2c16a40
to
2b33743
Compare
2b33743
to
fa7bb86
Compare
07b6392
to
9bda4f7
Compare
I think this should be done now. Just one thing: The style check currently fails because it is being tested against a new version of Ruff, which should be updated at our runtime in a different PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good!
@alissonlauffer in my opinion, it makes no sense to fallback to
The If I don't think it's a good choice to implicitly limit the allowed users to be only the bot that sent the keyboard if the user didn't specify anyone in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
9bda4f7
to
0671877
Compare
…}` methods and cleanup chat_id and user_id typing hints
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good, thank you my friend
The PR of Pyromod implementation (#1) was submitted before the change in organization and copyright holder. As a result, we forgot to update the holder information in the files from Amano LLC to Hydrogram.
* docs: fixed typos in the contributing guide * fix: tgcrypto link in speedups doc * refactor: improved sphinx configuration * refactor: changed copyright holder of pyromod The PR of Pyromod implementation (#1) was submitted before the change in organization and copyright holder. As a result, we forgot to update the holder information in the files from Amano LLC to Hydrogram. * docs: use correct category for news fragment 15 * Merge branch 'dev' of https://github.com/hydrogram/hydrogram into docs-improvements * chore: update furo to 2024.1.29 * docs: fixed a typo in github url * docs: disable `napoleon_preprocess_types` It was causing problems with the formatting of the types in the documentation pages * docs(contributing): improved feedback and commits section * Merge branch 'dev' of https://github.com/hydrogram/hydrogram into docs-improvements * fix(news): fixed a typo * chore: update sphinx-autobuild * docs: added mtproto-vs-bot-api image
Description
This PR integrates Pyromod patches into Hydrogram. The code has been reformatted and lined according to the project's style guidelines.
Type of change
Checklist