Skip to content

Update MqttServer.cs#1858

Merged
chkr1011 merged 2 commits intodotnet:masterfrom
kimdiego2098:patch-1
Nov 1, 2023
Merged

Update MqttServer.cs#1858
chkr1011 merged 2 commits intodotnet:masterfrom
kimdiego2098:patch-1

Conversation

@kimdiego2098
Copy link
Copy Markdown
Contributor

clientID is null => RetainedMessageChangedEventArgs ctor throw new ArgumentNullException(nameof(clientId)

kimdiego2098 and others added 2 commits October 16, 2023 11:38
clientID is null => RetainedMessageChangedEventArgs ctor throw new ArgumentNullException(nameof(clientId)
@chkr1011
Copy link
Copy Markdown
Collaborator

@kimdiego2098 Please accept the CLA so that I can merge the PR.

@kimdiego2098
Copy link
Copy Markdown
Contributor Author

@dotnet-policy-service agree

@kimdiego2098
Copy link
Copy Markdown
Contributor Author

@kimdiego2098 Please accept the CLA so that I can merge the PR.

is ok @chkr1011

@kimdiego2098
Copy link
Copy Markdown
Contributor Author

Is ci build failed the reason why pr cannot be merged? Can we release the version as soon as possible
@chkr1011

@chkr1011 chkr1011 merged commit e4f605f into dotnet:master Nov 1, 2023
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.

2 participants