-
Notifications
You must be signed in to change notification settings - Fork 342
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
1616 v4 catch up with v3 #1617
1616 v4 catch up with v3 #1617
Conversation
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.
I removed this, as we don't have any current plans to add a compatibility module. I can re-instate it if we decide we need it.
@@ -146,7 +146,7 @@ def __init__( | |||
execution_options_builder = ExecutionOptionsBuilder() | |||
execution_options_builder.connection_strategy = ConnectionStrategy.default() | |||
if endpoint_id is not None: | |||
execution_options_builder.connection_strategy(ConnectionStrategy.endpoint_id(endpoint_id)) | |||
execution_options_builder.connection_strategy = ConnectionStrategy.endpoint_id(endpoint_id) |
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.
squashing a pre-existing bug
) | ||
program = program.copy_everything_except_instructions() | ||
assert len(program.instructions) == 0 # the purpose of copy_everything_except_instructions() | ||
assert len(program.declarations) == 0 # this is a view on the instructions member; must be consistent |
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.
Description
Not many changes, despite how gnarly the conflicts were.