Skip to content

Conversation

arebya
Copy link
Contributor

@arebya arebya commented Aug 13, 2018

As mentioned in issue#757

We need a asReader(Charset charset) method in Response.Body

*/
public final class Response implements Closeable {

private final int status;
Copy link
Member

Choose a reason for hiding this comment

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

could you revert this format changes?

runninng mvn clean install should fix code format

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@velo velo added enhancement For recommending new capabilities waiting for feedback Issues waiting for a response from either to the author or other maintainers labels Aug 13, 2018
Copy link
Member

@velo velo left a comment

Choose a reason for hiding this comment

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

Please run mvn clean install to fix code format

@velo velo merged commit cb611ef into OpenFeign:master Aug 14, 2018
@kdavisk6 kdavisk6 mentioned this pull request Dec 5, 2018
velo pushed a commit that referenced this pull request Oct 7, 2024
* add charset to Response.Body when create Reader

* format

* support charset when build Reader for Response.Body

* support charset when build Reader for Response.Body

* support charset when build Reader for Response.Body

* format header

* format Response
velo pushed a commit that referenced this pull request Oct 8, 2024
* add charset to Response.Body when create Reader

* format

* support charset when build Reader for Response.Body

* support charset when build Reader for Response.Body

* support charset when build Reader for Response.Body

* format header

* format Response
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement For recommending new capabilities waiting for feedback Issues waiting for a response from either to the author or other maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants