-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Check that we have documented recent compiler messages #5410
Comments
Yes, here is the list for C#. It is largely chronological. |
@BillWagner I just tried to compare the pages under Click to expand the full table (more than 1500 rows)
|
@mikkelbu That's the plan @rpetrusha are working on putting together. We'll also have a similar project for VB. Stay tuned. We'd love your help. |
@pkulikov No, I forgot about that issue (eventhough I looked at it this week). Then it seems that this it is a smaller task, but still a large task IMHO. |
As I understand, the first priority is to document error messages. That can be a smaller task, while documenting all the compiler warnings might be an epic :) |
There's also #2363, which treats storing C# and VB error messages/warnings in a unified directory per language. Moving files from a |
I've assigned this to the current milestone, with the goal of having the following actions completed during this sprint:
@rpetrusha Comment on this plan, or add new tasks that you think are reasonable. |
This looks good, @BillWagner. I think that we can link to the project to track tasks from the community contributor's project. In the project to track tasks, we should also include #1491 on it. I think that that should be the first item of work. |
Messages are in the roslyn resource file |
I can't find the message for CS0043 that's already documented in that link. Even a search for it in the dotnet organization gives only the documentation result. A search in roslyn has no results too. @BillWagner , Do you know anything about that ? Update: Looking at ErrorCode.cs. Seems like this error isn't used by roslyn. I've opened #14737 for this specific one. |
This issue has been closed as part of the issue backlog grooming process outlined in #22351. That automated process may have closed some issues that should be addressed. If you think this is one of them, reopen it with a comment explaining why. Tag the |
@BillWagner You'll probably want to re-open this. |
Related to #24002 |
I'm almost certain we haven't added new warnings and error messages since C# 6.0 (and maybe earlier).
@jcouv is there a good list in the roslyn repo to track down what should be covered?
See #5390-comment for details.
The text was updated successfully, but these errors were encountered: