-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Docs link python classes to docs #2633 #2738
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
Docs link python classes to docs #2633 #2738
Conversation
Telegram changed these lines, so did we now.
…ryptionError (python-telegram-bot#2621) * move telegramdecryptionerror to error.py * Change error class name
…t#2612) * feat: add docs about docs * fix: improve looks * fix: make link work * fix: this looks better * Improved markdown, updated link * Less justifying Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
* Fix incomplete type annotations for CallbackContext Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
* Feat: Custom pytest marker Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
* Make basepersistence methods abstractmethod Signed-off-by: starry69 <starry369126@outlook.com> Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
…m-bot#2634) Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
# Conflicts: # telegram/bot.py
…ryptionError (python-telegram-bot#2621) * move telegramdecryptionerror to error.py * Change error class name
…t#2612) * feat: add docs about docs * fix: improve looks * fix: make link work * fix: this looks better * Improved markdown, updated link * Less justifying Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
* Fix incomplete type annotations for CallbackContext Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
* Feat: Custom pytest marker Co-authored-by: Bibo-Joshi <22366557+Bibo-Joshi@users.noreply.github.com>
* Make basepersistence methods abstractmethod Signed-off-by: starry69 <starry369126@outlook.com> Co-authored-by: Hinrich Mahler <22366557+Bibo-Joshi@users.noreply.github.com>
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com> Co-authored-by: poolitzer <25934244+Poolitzer@users.noreply.github.com>
# Conflicts: # telegram/bot.py # telegram/ext/callbackcontext.py # telegram/ext/dispatcher.py # telegram/ext/extbot.py # telegram/ext/jobqueue.py # tests/conftest.py # tests/test_bot.py
Thanks for the PR! one could think of linking to the docs of APScheduler for the JobQueue 🤔 BTW, would you mind rebasing your branch onto v14 to clean up the commit history? :D |
when i tried "rebase current onto selected", where "selected" is the v14 branch, i got this error message: i don't have any uncommitted changes, except for some odd reason the @Bibo-Joshi what should i do to resolve this? |
Please try to reset the changes in ptb_urllib3 as described in the contribution guide. An alternative to rebasing would be to create a new branch from the upt-to-date v14 branch and cherry pick your RTD-related commits into that branch. If neither of this works, let's discuss that in PM or in @pythontelegrambotdev :) |
ok i ran |
@eldbud search for "vendor" ;) |
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.
Hey there. Relax, I am just a little warning for the maintainers to release directly after merging your PR, otherwise we have broken examples and people might get confused :)
yes found it, ran the command, updated v14 from upstream, merged v14 into my branch, ran the Edit: I also now created a completely new branch from a freshly updated v14, and the urllib3 file is still in the changes list. why is this there? i'm lost here |
Also, now i see i have conflicting files with the base branch. do i need to merge the Edit: i merged |
ran with nitpicky, and it gives me many |
Resolves item number 2 on the original message in #2633 - link to python docs.
Added intersphinx to conf.py, also minor docs mending.
I am uncertain whether i should add more links to any additional libraries apart from the python builtin library. Please do tell if you can think of any library that we could benefit from including in the intersphinx.