Skip to content
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

nullable anotation Part 1 and new UndocumentedTags property #246

Merged
merged 2 commits into from
Sep 16, 2023

Conversation

AoshiW
Copy link

@AoshiW AoshiW commented Aug 7, 2023

  • adding nullable annotation for .Client.Models project (except: MessageEmote.cs, builders and all models based on USERNOTICE command)
  • added new UndocumentedTags property for most models
  • (temporary) suppression of some warning
    • CS1591 Missing XML comment for publicly visible type or member

part 2 will focus on .Client,

Copy link
Member

@swiftyspiffy swiftyspiffy left a comment

Choose a reason for hiding this comment

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

lgtm

{
public sealed class OutboundChatMessageBuilder : IBuilder<OutboundChatMessage>
{
private string _channel;
private string _message;
private string _userName;
Copy link
Member

Choose a reason for hiding this comment

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

so i know the username here seemed kind of redundant, but i believe it was added because there were users of the library that had a single OutboundChatMessage event for multiple bot instances using multiple users, so they wanted to know which account/bot was sending the outgoing message.

Not apposed to removing though.

@swiftyspiffy swiftyspiffy merged commit 56f89f5 into TwitchLib:dev Sep 16, 2023
1 check passed
@AoshiW AoshiW deleted the nullable branch October 4, 2023 06:34
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.

3 participants