Skip to content

Made Request.prepare private (i.e. renamed it to _prepare) - #1280

Closed
cdeler wants to merge 3 commits into
encode:masterfrom
cdeler:tightening-public-API-on-models
Closed

Made Request.prepare private (i.e. renamed it to _prepare)#1280
cdeler wants to merge 3 commits into
encode:masterfrom
cdeler:tightening-public-API-on-models

Conversation

@cdeler

Copy link
Copy Markdown
Contributor

I created a PR regarding to https://github.com/encode/httpx/issues/1275

I renamed Request.prepare to _prepare making this method private

@cdeler
cdeler marked this pull request as draft September 11, 2020 09:18
@cdeler
cdelerforce-pushed the tightening-public-API-on-models branch from e254aa3 to 9adc8d9CompareSeptember 11, 2020 16:04
@cdeler
cdeler marked this pull request as ready for review September 11, 2020 16:07
@StephenBrown2

Copy link
Copy Markdown
Contributor

This has a lot of changes unrelated to the title, can you back those out and submit them as separate PRs?

@cdeler

cdeler commented Sep 11, 2020

Copy link
Copy Markdown
ContributorAuthor

Sure, thank you. I'm marking that as a draft

I had had a glitch with pipeline, I thought that making this pr "ready" could solve something)

@cdeler
cdeler marked this pull request as draft September 11, 2020 17:44
@cdelercdeler mentioned this pull request Sep 11, 2020
@cdeler

Copy link
Copy Markdown
ContributorAuthor

Closing the PR, it will be split

@cdelercdeler closed this Sep 11, 2020
@cdeler
cdeler deleted the tightening-public-API-on-models branch September 11, 2020 19:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@cdeler@StephenBrown2