-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add template types to Connection
#6709
Add template types to Connection
#6709
Conversation
1e58533
to
cded708
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where did those abstract functions go?
They are actually defined by -- |
I'll fix the conflicts once #6713 is merged. |
Thanks for explaining (and extracting). |
8214216
to
62d5cf4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing cleanup of the baseline!
Description
Making
BaseConnection
a generic type, thus extensions need to specify the type of the connection.Checklist: