Skip to content

Introduce the page for CS8347 - #55940

Merged
BillWagner merged 4 commits into
dotnet:mainfrom
BartoszKlonowski:feat/cs8347/new-page
Sep 11, 2026
Merged

Introduce the page for CS8347 #55940
BillWagner merged 4 commits into
dotnet:mainfrom
BartoszKlonowski:feat/cs8347/new-page

Conversation

@BartoszKlonowski

@BartoszKlonowski BartoszKlonowski commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

This pull request creates the initial page for CS8347 error, with the explanation of the root cause, example and two ways of how to correct the error.

As operating with reference requires a specific mindset, my wording on that may not be perfect when it comes to explaining why CS8347 may in fact be an issue - I tried to describe it best according to my dictionary.
In case of an idea for better explanation all comments are welcome :)


Internal previews

File Preview link
docs/csharp/language-reference/compiler-messages/cs8347.md Learn preview
docs/csharp/language-reference/toc.yml Learn preview
docs/csharp/misc/sorry-we-don-t-have-specifics-on-this-csharp-error.md Learn preview

Build report

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @BartoszKlonowski

This LGTM and I'll :shipit: now.

@BillWagner
BillWagner enabled auto-merge (squash) September 11, 2026 14:06
@BillWagner

Copy link
Copy Markdown
Member

@BartoszKlonowski Can you fix the lint issues? Then, this is ready.

auto-merge was automatically disabled September 11, 2026 15:19

Head branch was pushed to by a user without write access

@BartoszKlonowski

BartoszKlonowski commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

@BillWagner Done!
And thank you for the review 👍

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks again @BartoszKlonowski

@BillWagner
BillWagner merged commit c2bef19 into dotnet:main Sep 11, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants