Skip to content

Conversation

@ParadaCarleton
Copy link

No description provided.

@ParadaCarleton
Copy link
Author

bors try

bors bot added a commit that referenced this pull request Feb 15, 2022
@bors
Copy link
Contributor

bors bot commented Feb 15, 2022

try

Build failed:

Copy link
Member

@devmotion devmotion left a comment

Choose a reason for hiding this comment

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

Thanks! I think you have to delete the existing constructors. Can you add tests for all constructors as well and update the version number?

Carlos Parada and others added 2 commits February 15, 2022 11:49
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@ParadaCarleton
Copy link
Author

Thanks! I think you have to delete the existing constructors. Can you add tests for all constructors as well and update the version number?

Sure!

@ParadaCarleton
Copy link
Author

bors try

bors bot added a commit that referenced this pull request Feb 15, 2022
Carlos Parada and others added 2 commits February 15, 2022 13:01
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@devmotion
Copy link
Member

@ParadaCarleton With bors we don't have to test every single commit but can wait until the format is correct and (usually) until the design of the PR is approved.

Carlos Parada and others added 2 commits February 15, 2022 13:14
Co-authored-by: David Widmann <devmotion@users.noreply.github.com>
Carlos Parada and others added 2 commits February 15, 2022 14:49
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented Feb 15, 2022

try

Build failed:

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: David Widmann <devmotion@users.noreply.github.com>
@ParadaCarleton
Copy link
Author

cool; should I merge @devmotion ?

Copy link
Member

@devmotion devmotion left a comment

Choose a reason for hiding this comment

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

LGTM!

@ParadaCarleton
Copy link
Author

bors r+

bors bot pushed a commit that referenced this pull request Feb 17, 2022
Co-authored-by: David Widmann <devmotion@users.noreply.github.com>
@bors
Copy link
Contributor

bors bot commented Feb 17, 2022

Timed out.

@yebai
Copy link
Member

yebai commented Feb 17, 2022

All the tests passed - it's unclear why bors timed out. I'll merge this PR manually.

@yebai yebai merged commit c6ccb08 into master Feb 17, 2022
@yebai yebai deleted the Extra-Context-constructors branch February 17, 2022 15:30
@yebai
Copy link
Member

yebai commented Feb 17, 2022

thanks @ParadaCarleton!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants