Skip to content

Conversation

crepererum
Copy link
Collaborator

Small clean-up before I start to work on #182.

Copy link
Contributor

@domodwyer domodwyer left a comment

Choose a reason for hiding this comment

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

Accidental mut?

@crepererum
Copy link
Collaborator Author

Accidental mut?

Semi. It is mutable while bootstrapping the connection, but during bootstrap we also have a mutable reference to the Messenger. The versions should not change/re-sync while the connection is in use.

@crepererum crepererum added the automerge Instruct kodiak to merge the PR label Oct 27, 2022
@kodiakhq kodiakhq bot merged commit 12c8d4c into main Oct 27, 2022
@kodiakhq kodiakhq bot deleted the crepererum/issue182a branch October 27, 2022 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Instruct kodiak to merge the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants