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

[compat] Enforce CS1591 to document all public APIs in the repo #46283

Open
carlossanlop opened this issue Jan 24, 2025 · 3 comments
Open

[compat] Enforce CS1591 to document all public APIs in the repo #46283

carlossanlop opened this issue Jan 24, 2025 · 3 comments
Labels
Area-ApiCompat untriaged Request triage from a team member

Comments

@carlossanlop
Copy link
Member

CS1591 has severity set to suggestion in the root .editorconfig:

dotnet_diagnostic.CS1591.severity = suggestion

We should change it to warning so that public APIs are triple-slash documented.

There might be other infra configuration changes to ensure docs work properly at the repo level.

@marcpopMSFT
Copy link
Member

Is this for the api compat area in particular? The rest of the SDK doesn't release packages on nuget.org for consumption so doesn't have an api surface area intended to be used.

@carlossanlop
Copy link
Member Author

Compatibility. Let me change the title.

@carlossanlop carlossanlop changed the title Enforce CS1591 to document all public APIs in the repo [compat] Enforce CS1591 to document all public APIs in the repo Jan 28, 2025
Copy link
Contributor

@dotnet/area-infrastructure-libraries a new issue has been filed in the ApiCompat area, please triage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-ApiCompat untriaged Request triage from a team member
Projects
None yet
Development

No branches or pull requests

2 participants