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

Add readme to nuget packages #121

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

FroggieFrog
Copy link

This PR adds the already existing readme files of the various packages to those packages.
Currently only the description is displayed in the official gallery.

example images (from BaGetter gallery)

before

grafik

after

grafik

@@ -4,6 +4,7 @@

<PackageTags>NuGet</PackageTags>
<Description>The core libraries that power BaGetter.</Description>
<PackageReadmeFile>readme.md</PackageReadmeFile>

Choose a reason for hiding this comment

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

I don't think this is always the right way. BaGetter.Core has a readme describing the source folder contents, which does not fit as package description on the gallery page. I'm not sure how to handle this though, since the readme here does fit a github page, describing its contents. Maybe an additional PackageReadme.md file for information to display on the gallery page?

<PackageTags>NuGet</PackageTags>
<Description>The libraries to host BaGetter on MySQL.</Description>
<PackageReadmeFile>readme.md</PackageReadmeFile>

Choose a reason for hiding this comment

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

The db provider projects have no fitting readme files either. They just describe how to add a migration for each db provider.

@FroggieFrog FroggieFrog marked this pull request as draft March 19, 2024 12:39
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.

2 participants