Skip to content

Fix failing imports from httpcore#5891

Merged
CirqBot merged 3 commits intoquantumlib:masterfrom
pavoljuhas:update-httpcore-imports
Sep 27, 2022
Merged

Fix failing imports from httpcore#5891
CirqBot merged 3 commits intoquantumlib:masterfrom
pavoljuhas:update-httpcore-imports

Conversation

@pavoljuhas
Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas commented Sep 27, 2022

Refactor test_rigetti_qcs_service_api_call() to use public API from httpx instead
of private API from httpcore which changed.

Also require pyquil>=3.2.0 to ensure httpx and httpcore include API change at httpcore==0.14.0,
encode/httpcore@f9b9391.

Note: pyquil creates indirect dependency on httpcore as follows:

  • pyquil (==3.2.0) requires qcs-api-client (>=0.20.13,<0.22.0)
  • qcs-api-client (==0.21.2) requires httpx (>=0.23.0,<0.24.0)
  • httpx (==0.23.0) requires httpcore (>=0.15.0,<0.16.0)

The `httpcore._types` module was removed in
encode/httpcore@f9b9391.
Indirect dependency chain on httpcore:

- pyquil (==3.2.0) requires qcs-api-client (>=0.20.13,<0.22.0)
- qcs-api-client (==0.21.2) requires (httpx (>=0.23.0,<0.24.0)
- httpx (==0.23.0) requires httpcore (>=0.15.0,<0.16.0)
@pavoljuhas pavoljuhas requested review from a team, cduck and vtomole as code owners September 27, 2022 00:28
@CirqBot CirqBot added the Size: XS <10 lines changed label Sep 27, 2022
@CirqBot CirqBot added the size: S 10< lines changed <50 label Sep 27, 2022
@pavoljuhas pavoljuhas changed the title Fix failing imports from recent httpcore Fix failing imports from httpcore Sep 27, 2022
Copy link
Copy Markdown
Collaborator

@tanujkhattar tanujkhattar left a comment

Choose a reason for hiding this comment

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

Thanks for debugging this!

cc @erichulburd

@pavoljuhas pavoljuhas added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Sep 27, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Sep 27, 2022
@CirqBot CirqBot merged commit 490017a into quantumlib:master Sep 27, 2022
@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Sep 27, 2022
@pavoljuhas pavoljuhas deleted the update-httpcore-imports branch September 27, 2022 17:13
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Refactor `test_rigetti_qcs_service_api_call()` to use public API from httpx instead
of private API from httpcore which changed.

Also require pyquil>=3.2.0 to ensure httpx and httpcore include API change at httpcore==0.14.0,
encode/httpcore@f9b9391.

Note: pyquil creates indirect dependency on httpcore as follows:

- pyquil (==3.2.0) requires qcs-api-client (>=0.20.13,<0.22.0)
- qcs-api-client (==0.21.2) requires httpx (>=0.23.0,<0.24.0)
- httpx (==0.23.0) requires httpcore (>=0.15.0,<0.16.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50 Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants