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

System.Data.SqlClient is missing NuGet package README file #86661

Open
mckennabarlow opened this issue May 23, 2023 · 5 comments
Open

System.Data.SqlClient is missing NuGet package README file #86661

mckennabarlow opened this issue May 23, 2023 · 5 comments
Labels
area-System.Data.SqlClient untriaged New issue has not been triaged by the area owner

Comments

@mckennabarlow
Copy link

As one of the top most installed NuGet packages, your System.Data.SqlClient package is extremely important to the developer ecosystem. The NuGet team has noticed that your package on NuGet.org is missing a README file. Please take a moment to add a README file to your package repository.

Our customer research indicates that one of the top problems that package consumers face is insufficient package documentation such as README files. Adding a README file to your package will help users quickly understand what the package is and what it does. Since your README will be the first impression for users when they view your package on NuGet.org, it is crucial for authors to write and include high-quality READMEs for their packages.

There is no specific format or structure to a good README – instead it’s about effectively communicating the purpose and functionality of your package to potential users. If you are unsure of where to start, check out our blog post for some README best practices and a template file.
If you are new to NuGet README, learn more about how to add a README to your NuGet Package.

Your feedback is very important to us. If you experience any issue using READMEs or have ideas for improvements – please feel free to contact us by replying to this issue

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 23, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label May 23, 2023
@ghost
Copy link

ghost commented May 23, 2023

Tagging subscribers to this area: @DavoudEshtehari, @David-Engel, @JRahnama
See info in area-owners.md if you want to be subscribed.

Issue Details

As one of the top most installed NuGet packages, your System.Data.SqlClient package is extremely important to the developer ecosystem. The NuGet team has noticed that your package on NuGet.org is missing a README file. Please take a moment to add a README file to your package repository.

Our customer research indicates that one of the top problems that package consumers face is insufficient package documentation such as README files. Adding a README file to your package will help users quickly understand what the package is and what it does. Since your README will be the first impression for users when they view your package on NuGet.org, it is crucial for authors to write and include high-quality READMEs for their packages.

There is no specific format or structure to a good README – instead it’s about effectively communicating the purpose and functionality of your package to potential users. If you are unsure of where to start, check out our blog post for some README best practices and a template file.
If you are new to NuGet README, learn more about how to add a README to your NuGet Package.

Your feedback is very important to us. If you experience any issue using READMEs or have ideas for improvements – please feel free to contact us by replying to this issue

Author: mckennabarlow
Assignees: -
Labels:

area-System.Data.SqlClient, untriaged, needs-area-label

Milestone: -

@filipnavara filipnavara removed the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 23, 2023
@filipnavara
Copy link
Member

cc @ericstj @ViktorHofer Not sure what's the correct area for this issue. The package is effectively replaced by Microsoft.Data.SqlClient and I am not sure what's the official support status for the old System.Data.SqlClient one.

@ericstj
Copy link
Member

ericstj commented May 23, 2023

Readme's can be added without updating the package. The content can be added manually via
https://www.nuget.org/packages/System.Data.SqlClient/4.8.5/Manage

@dotnet/sqlclientdevteam if you wanted to provide a readme for the old SqlClient package we can help you make the change in Nuget.org. The same change could be made to the package in the old servicing branches - should it ever need to be built again.

@JonDouglas
Copy link

@ericstj That might be referring to the old "readme.txt" experience. For the new README.md experience, the readme is included with the package itself. If a package cannot be updated/built we should just ignore them for now.

The primary case here is to provide consumers with enough information to get started with these packages. Here is an example:

https://www.nuget.org/profiles/dotnetframework

Take System.Buffers for example:

image (no readme is rendered as only a description is today)

vs.

image

In an upcoming Visual Studio and Visual Studio Code release, we will be showing READMEs in the NuGet package manager UI experience and thus would love to help developers get started with some of the most popular packages. Also given the visibility of .NET packages, this sets a positive example on the entire ecosystem to add READMEs organically (many OSS projects already do this today).

One goal for .NET 8 would be to rally around the most used and "active" packages to get READMEs included. If someone can help our team & the .NET release team with an inventory of what packages may be good candidates for READMEs, we can start to piece-meal some READMEs for them or get some documentation team help to do so.

i.e. Each consumer-friendly package could supplement README information from the respective official docs:

i.e. https://learn.microsoft.com/en-us/dotnet/standard/io/buffers

@ericstj ericstj changed the title Missing NuGet package README file System.Data.SqlClient is missing NuGet package README file Jul 25, 2023
@ericstj
Copy link
Member

ericstj commented Jul 25, 2023

If a package cannot be updated/built we should just ignore them for now.

Are you saying that if we make changes in https://www.nuget.org/packages/System.Data.SqlClient/4.8.5/Manage to update the readme and discourage use of old packages that NuGet wouldn't present those changes to the user?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Data.SqlClient untriaged New issue has not been triaged by the area owner
Projects
None yet
Development

No branches or pull requests

4 participants