Skip to content

[HttpClient] Remove deprecated usage of GuzzleHttp\Promise\queue #44890

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

Merged
merged 1 commit into from
Jan 12, 2022
Merged

[HttpClient] Remove deprecated usage of GuzzleHttp\Promise\queue #44890

merged 1 commit into from
Jan 12, 2022

Conversation

GrahamCampbell
Copy link
Contributor

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

GuzzleHttp\Promise\queue was deprecated in 1.4.0 in favour of GuzzleHttp\Promise\Utils::queue, to enable PSR-4 compliance and eliminate issues with multiple installs globally and locally. Since the minimum version of this package in Symfony's composer.json is 1.4.0 (^1.4), then this change should be safe to make.


cc @Nyholm

@carsonbot carsonbot added this to the 5.4 milestone Jan 2, 2022
@carsonbot carsonbot changed the title Remove deprecated usage of GuzzleHttp\Promise\queue [HttpClient] Remove deprecated usage of GuzzleHttp\Promise\queue Jan 2, 2022
@derrabus derrabus modified the milestones: 5.4, 4.4 Jan 2, 2022
@derrabus
Copy link
Member

derrabus commented Jan 2, 2022

Thank you! This should be fixed on the 4.4 branch, I suppose. Also, please make fabbot happy. 🙂

@nicolas-grekas nicolas-grekas changed the base branch from 5.4 to 4.4 January 12, 2022 18:29
@nicolas-grekas
Copy link
Member

Thank you @GrahamCampbell.

@nicolas-grekas nicolas-grekas merged commit 9f5238d into symfony:4.4 Jan 12, 2022
@GrahamCampbell GrahamCampbell deleted the fix-depr branch January 12, 2022 19:48
nicolas-grekas added a commit that referenced this pull request Jan 13, 2022
…\promise_for (plozmun)

This PR was merged into the 5.3 branch.

Discussion
----------

[HttpClient] Remove deprecated usage of GuzzleHttp\Promise\promise_for

| Q             | A
| ------------- | ---
| Branch?       | 5.3,
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Similar to #44890 but this function was introduced in 5.1 #37927

Commits
-------

3d2ed70 [HttpClient] Remove deprecated usage of GuzzleHttp\Promise\promise_for
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants