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

Including README.md in the NuGet Packages #175

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Conversation

DennisDyallo
Copy link
Collaborator

@DennisDyallo DennisDyallo commented Dec 17, 2024

Description

Now including README.md in the NuGet packages, which creates a better first impression of the project.

Why include the README?

image

Key changes in the README

Here are the key changes made to improve the README.md:

Table of Contents

  • Fixed anchor links by using GitHub-compatible formatting
  • Added nested structure for better navigation
  • Added missing sections

Content Organization

  • Added Basic Usage section with practical code example
  • Streamlined the introduction
  • Removed redundant content

Formatting

  • Added emoji icons for better visual hierarchy
  • Consistent heading levels
  • Clean spacing and indentation

Structure

  • Added clear sections for Installation and Basic Usage
  • Simplified the Security section
  • Added direct issue creation link

Removed

  • Redundant documentation explanations
  • Extra whitespace
  • Unnecessary technical details

The main focus was on making the document more navigable and user-friendly while maintaining all essential information.

@DennisDyallo DennisDyallo changed the title Updated README.md for better readability Including README.md in the NuGet Packages Dec 17, 2024
Copy link

Test Results: Windows

    2 files      2 suites   5s ⏱️
3 741 tests 3 741 ✅ 0 💤 0 ❌
3 743 runs  3 743 ✅ 0 💤 0 ❌

Results for commit 1e55362.

Copy link

Test Results: Ubuntu

    2 files      2 suites   10s ⏱️
3 733 tests 3 733 ✅ 0 💤 0 ❌
3 735 runs  3 735 ✅ 0 💤 0 ❌

Results for commit 1e55362.

Copy link

Test Results: MacOS

    2 files      2 suites   4s ⏱️
3 733 tests 3 733 ✅ 0 💤 0 ❌
3 735 runs  3 735 ✅ 0 💤 0 ❌

Results for commit 1e55362.

Copy link

Code Coverage

Package Line Rate Branch Rate Complexity Health
Yubico.Core 44% 32% 4311
Yubico.YubiKey 49% 46% 19671
Summary 48% (32918 / 68499) 43% (8245 / 18990) 23982

Minimum allowed line rate is 40%

@DennisDyallo
Copy link
Collaborator Author

fyi @dainnilsson

@DennisDyallo DennisDyallo merged commit 65f98c5 into develop Dec 17, 2024
11 checks passed
@DennisDyallo DennisDyallo deleted the docs/packagereadme branch December 17, 2024 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant