NetCordDev / NetCord

The modern and fully customizable C# Discord library.
https://netcord.dev
MIT License
65 stars 10 forks source link

Added missing flags and documentation to 'UserFlags.cs'. #17

Closed Red-K0 closed 5 months ago

KubaZ2 commented 5 months ago

The names should be PascalCase. Could you also replace spaces before and after the summary content with new lines and remove them after casting?

Red-K0 commented 5 months ago

The names should be PascalCase. Could you also replace spaces before and after the summary content with new lines and remove them after casting?

Yeah, all done.

KubaZ2 commented 5 months ago

Could you also remove the 0 prefixes from bitfield values? They look odd for me. Also could you please split the values with new lines? I think it would make the code more readable.

Red-K0 commented 5 months ago

Could you also remove the 0 prefixes from bitfield values? They look odd for me. Also could you please split the values with new lines? I think it would make the code more readable.

Split and prefixes removed

KubaZ2 commented 5 months ago

Could you also remove the empty spaces? I think it should be Name = value for all fields. I think the unnecessary indentation is useless.

Red-K0 commented 5 months ago

Could you also remove the empty spaces? I think it should be Name = value for all fields. I think the unnecessary indentation is useless.

Yeah that's more a remnant of how the code originally looked, removing it

Red-K0 commented 5 months ago

Not sure why it didn't go through the first time, but extra space is removed

KubaZ2 commented 5 months ago

I reminded myself that https://github.com/KubaZ2/NetCord/blob/alpha/NetCord%2FPermissions.cs are formatted with uL suffix instead of casting. Could you follow that convention?

Red-K0 commented 5 months ago

uL convention added

KubaZ2 commented 5 months ago

Could you also fix casing of private when it follows Undocumented and?

Red-K0 commented 5 months ago

Sorry for the delay, casings fixed