Fixed Edit on github links for v13.x documentation by javad94 · Pull Request #3164 · python-telegram-bot/python-telegram-bot · GitHub
Skip to content

Fixed Edit on github links for v13.x documentation - #3164

Merged
harshil21 merged 6 commits into
python-telegram-bot:doc-fixes-v13from
javad94:13.x-doc-fix
Aug 1, 2022
Merged

Fixed Edit on github links for v13.x documentation #3164
harshil21 merged 6 commits into
python-telegram-bot:doc-fixes-v13from
javad94:13.x-doc-fix

Conversation

@javad94

@javad94 javad94 commented Jul 22, 2022

Copy link
Copy Markdown

Checklist for PRs

  • Added .. versionadded:: version, .. versionchanged:: version or .. deprecated:: version to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)
  • Created new or adapted existing unit tests
  • Documented code changes according to the CSI standard
  • Added myself alphabetically to AUTHORS.rst (optional)
  • Added new classes & modules to the docs and all suitable __all__ s

@javad94

javad94 commented Jul 22, 2022

Copy link
Copy Markdown
Author

harshil21
harshil21 approved these changes Jul 23, 2022
View reviewed changes

@harshil21 harshil21 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the fix. Is your sphinx version the one specified in requirements-docs.txt? The error probably happens since the versions aren't exactly matching the ones we specified.

@javad94

javad94 commented Jul 23, 2022

Copy link
Copy Markdown
Author

Thanks for the fix. Is your sphinx version the one specified in requirements-docs.txt? The error probably happens since the versions aren't exactly matching the ones we specified.

Yes, it is the same version.
image

@harshil21

harshil21 commented Jul 23, 2022

Copy link
Copy Markdown
Member

okay since I see you're on a venv you probably have the right versions I'd say to go ahead and add that in (Jinja2 < 3.1). Makes sense you're getting this since I see this in the sphinx source code.

@javad94

javad94 commented Jul 23, 2022

Copy link
Copy Markdown
Author

okay since I see you're on a venv you probably have the right versions I'd say to go ahead and add that in (Jinja2 < 3.1). Makes sense you're getting this since I see this in the sphinx source code.

Of course, done.

harshil21
harshil21 reviewed Jul 23, 2022
View reviewed changes
Comment thread docs/requirements-docs.txt
@harshil21 harshil21 added the ⚙️ documentation affected functionality: documentation label Jul 23, 2022
@harshil21 harshil21 added this to the v13.14 milestone Jul 23, 2022
harshil21
harshil21 reviewed Jul 23, 2022
View reviewed changes
Comment thread docs/requirements-docs.txt Outdated
@javad94 @harshil21
Put comment link in newline
251f8cf 
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
@harshil21
harshil21 changed the base branch from v13.x to doc-fixes-v13 July 24, 2022 16:08
@javad94

javad94 commented Aug 1, 2022

Copy link
Copy Markdown
Author

@harshil21 Anything else needed to merge this PR?

@harshil21
harshil21 merged commit 864ee7a into python-telegram-bot:doc-fixes-v13 Aug 1, 2022
@harshil21

harshil21 commented Aug 1, 2022

Copy link
Copy Markdown
Member

Thanks for the contribution!

@javad94

javad94 commented Aug 1, 2022

Copy link
Copy Markdown
Author

@Bibo-Joshi Bibo-Joshi mentioned this pull request Aug 1, 2022
Merged
2 tasks
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@harshil21 harshil21 harshil21 approved these changes

Assignees

No one assigned

Labels

⚙️ documentation affected functionality: documentation

Projects

None yet

Milestone

v13.14

Development

Successfully merging this pull request may close these issues.

3 participants

@javad94 @harshil21 @Bibo-Joshi