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

Rectify Client#fromHttpApp #7148

Merged
merged 1 commit into from
Jun 12, 2023
Merged

Conversation

danicheg
Copy link
Member

No description provided.

@armanbilge
Copy link
Member

Did I screw up the merge? Very sorry 😢

@danicheg
Copy link
Member Author

Truth be told, it's me who approved your PR 😄
Also, we didn't release it, so no problem at all.

@armanbilge
Copy link
Member

we didn't release it

😶‍🌫️ don't say the "release" word for main branch ...

Copy link
Member

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

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

Thank you. Sad reminder that merges are going to get harder and harder 😢

@danicheg
Copy link
Member Author

Sad reminder that merges are going to get harder and harder

I was recently thinking of how it's hard could be to migrate from 0.23 to 1.0 for projects that hugely rely on http4s if there are no http4s contributors in the team. Certainly, it's not like CE 2 -> 3 migration but still.

@danicheg danicheg merged commit 416dd46 into http4s:main Jun 12, 2023
13 checks passed
@danicheg danicheg deleted the client-fromHttpApp branch June 12, 2023 10:30
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.

None yet

2 participants