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

Fix the warning output for the CS0078 rule #1171

Merged
merged 2 commits into from
Sep 8, 2024
Merged

Fix the warning output for the CS0078 rule #1171

merged 2 commits into from
Sep 8, 2024

Conversation

rstm-sf
Copy link
Contributor

@rstm-sf rstm-sf commented Sep 7, 2024

After the source code for C# is generated, similar warnings may appear when building the project with the generated code

warning CS0078: The 'l' suffix is easily confused with the digit '1' -- use 'L' for clarity

I suggest changing the literal (currently), or like the others, via disable/restore warning

Fix #1078

@mgravell mgravell merged commit 21d5c30 into protobuf-net:main Sep 8, 2024
1 check passed
@rstm-sf rstm-sf deleted the issues/1078 branch September 8, 2024 08:19
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.

CS0078 is reported for literal suffixes created by protogen
2 participants