Skip to content
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

Update requests links #10482

Closed
wants to merge 2 commits into from
Closed

Update requests links #10482

wants to merge 2 commits into from

Conversation

AA-Turner
Copy link
Member

Fixes CI due to the python-requests domain no longer working.

@tk0miya please can this go into 5.0.0 to fix CI / for correct docs?

Feature or Bugfix

  • Bugfix

A

@AA-Turner AA-Turner added this to the 5.0.0 milestone May 27, 2022
@@ -109,7 +109,6 @@

intersphinx_mapping = {
'python': ('https://docs.python.org/3/', None),
'requests': ('https://docs.python-requests.org/en/latest/', None),
Copy link
Member

Choose a reason for hiding this comment

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

Why removing this?

Copy link
Member Author

Choose a reason for hiding this comment

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

As I made the authentication link inline, and nothing else used interphinx requests.

A

Copy link
Member

Choose a reason for hiding this comment

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

I feel it's better to update this URL by new one (refs: #10484). What do you think?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't mind either way, just important for CI to be fixed.

A

Copy link
Member

Choose a reason for hiding this comment

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

Thanks, I'd like to choose #10484.

@tk0miya tk0miya closed this May 28, 2022
@AA-Turner AA-Turner deleted the requests branch June 16, 2022 22:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants