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

Deprecate HttpResponse.from() methods #5075

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

minwoox
Copy link
Contributor

@minwoox minwoox commented Jul 26, 2023

Motivation:
We have decided to deprecate HttpResponse.from(...) methods and use HttpResponse.of(...) instead for consistency. #4995 (comment)

Modification:

  • Deprecate HttpResponse.from(CompletionStage) and its variants.
    • HttpResponse.of(CompletionStage) and its variants are added.

Result:

  • HttpResponse.from(CompletionStage) and its variants methods are deprecated.
    • Use HttpResponse.of(CompletionStage) and its variants instead.

Motivation:
We have decided to deprecate `HttpResponse.from(...)` methods and use `HttpResponse.of(...)` instead for consistency.
line#4995 (comment)

Modification:
- Deprecate `HttpResponse.from(CompletionStage)` and its variants.
  - `HttpResponse.of(CompletionStage)` and its variants are added.

Result:
- `HttpResponse.from(CompletionStage)` and its variants methods are deprecated.
  - Use `HttpResponse.of(CompletionStage)` and its variants instead.
Copy link
Contributor

@jrhee17 jrhee17 left a comment

Choose a reason for hiding this comment

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

Checked that changes in HttpResponse looks good 👍
I assume the other changes are just IntelliJ replacing/reformatting. Thanks @minwoox 🙇 👍 🙇

Copy link
Member

@trustin trustin 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 cleaning this up, @minwoox!

Copy link
Contributor

@ikhoon ikhoon left a comment

Choose a reason for hiding this comment

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

Thanks!

@ikhoon ikhoon added this to the 1.25.0 milestone Jul 31, 2023
@ikhoon ikhoon merged commit 4edff55 into line:main Jul 31, 2023
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