Skip to content

Make public class public constructable#154

Merged
Lukasa merged 1 commit into
swift-server:masterfrom
krzyzanowskim:marcin/public-delegate-initializer
Jan 19, 2020
Merged

Make public class public constructable#154
Lukasa merged 1 commit into
swift-server:masterfrom
krzyzanowskim:marcin/public-delegate-initializer

Conversation

@krzyzanowskim

Copy link
Copy Markdown
Contributor

HTTPClientCopyingDelegate meant to be public, hence make the initializer public.

@swift-server-bot

Copy link
Copy Markdown

Can one of the admins verify this patch?

2 similar comments
@swift-server-bot

Copy link
Copy Markdown

Can one of the admins verify this patch?

@swift-server-bot

Copy link
Copy Markdown

Can one of the admins verify this patch?

@krzyzanowskim krzyzanowskim changed the title Make public class public contrictible Make public class public constructable Jan 19, 2020
@krzyzanowskim krzyzanowskim requested a review from Lukasa January 19, 2020 11:39
@weissi weissi requested a review from artemredkin January 19, 2020 20:13

@weissi weissi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to me, @artemredkin ?

@artemredkin

Copy link
Copy Markdown
Collaborator

Looks good, thank you!

@Lukasa

Lukasa commented Jan 19, 2020

Copy link
Copy Markdown
Collaborator

@swift-server-bot test this please

@Lukasa Lukasa merged commit d5aa3a1 into swift-server:master Jan 19, 2020
@weissi weissi added the 🆕 semver/minor Adds new public API. label Jan 19, 2020
@weissi weissi added this to the 1.1.0 milestone Jan 19, 2020
@Lukasa Lukasa mentioned this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🆕 semver/minor Adds new public API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants