Skip to content

Add README files and enable XML documentation for all NuGet packages#770

Closed
Copilot wants to merge 6 commits intomainfrom
copilot/fix-838c609e-2773-4068-b012-87d60b4350bd
Closed

Add README files and enable XML documentation for all NuGet packages#770
Copilot wants to merge 6 commits intomainfrom
copilot/fix-838c609e-2773-4068-b012-87d60b4350bd

Conversation

Copy link
Contributor

Copilot AI commented Aug 16, 2025

This PR addresses the missing README files and XML documentation for NuGet packages by adding comprehensive package documentation and enabling XML doc generation across all Reqnroll packages.

Changes Made

README Files Added

Created README.md files for all NuGet packages with package-specific content:

  • Core packages: Reqnroll.Parser, Reqnroll.Generator, Reqnroll.Utils, Reqnroll.Tools.MsBuild.Generation
  • Test framework packages: Reqnroll.MSTest, Reqnroll.NUnit, Reqnroll.xUnit, Reqnroll.TUnit
  • Plugin packages: Reqnroll.Autofac, Reqnroll.Windsor, Reqnroll.Microsoft.Extensions.DependencyInjection, Reqnroll.ExternalData, Reqnroll.Verify, Reqnroll.SpecFlowCompatibility, Reqnroll.CustomPlugin
  • Template package: Reqnroll.Templates.DotNet

Each README includes:

  • Package description and purpose with links to relevant external documentation
  • Key features and capabilities
  • Installation instructions using PowerShell syntax
  • Consistent documentation links with readable display text

XML Documentation Enabled

Added <GenerateDocumentationFile>true</GenerateDocumentationFile> to all package project files with <NoWarn>$(NoWarn);1591</NoWarn> to suppress missing XML comment warnings while still generating documentation files.

Package Configuration

  • SDK-style projects: Added <PackageReadmeFile>README.md</PackageReadmeFile> and included README.md files in packages
  • Nuspec-based projects: Added <readme>README.md</readme> in metadata and included README.md files in the files section

Documentation Improvements

  • Added external links with readable display text for related technologies (Microsoft DI, MSTest, NUnit, xUnit, TUnit, Verify, Castle Windsor, Autofac)
  • Linked to specific Reqnroll documentation sections (SpecFlow migration guide, generator configuration)
  • Standardized link formatting with consistent "Reqnroll Documentation" terminology
  • Removed pronunciation guide from all packages except the main Reqnroll package

Packages Previously Complete

  • Reqnroll: Already had PackageReadme.md and XML docs enabled
  • Reqnroll.Assist.Dynamic: Already had a README.md file

The changes ensure all Reqnroll NuGet packages follow modern packaging standards and provide comprehensive, accessible documentation for developers.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

…nroll.Generator, Reqnroll.Utils, Reqnroll.Tools.MsBuild.Generation

Co-authored-by: 304NotModified <5808377+304NotModified@users.noreply.github.com>
@304NotModified 304NotModified added the documentation Improvements or additions to documentation label Aug 16, 2025
Copilot AI and others added 2 commits August 16, 2025 22:26
…nit, xUnit, TUnit

Co-authored-by: 304NotModified <5808377+304NotModified@users.noreply.github.com>
…, DI, ExternalData, Verify, SpecFlowCompatibility, CustomPlugin, Templates

Co-authored-by: 304NotModified <5808377+304NotModified@users.noreply.github.com>
Copilot AI changed the title [WIP] Create a readme for each nuget package and add it to the package. Also enable the xml docs for the nuget package. Skip readmes for packages which has aleady a readme. Add README files and enable XML documentation for all NuGet packages Aug 16, 2025
Copilot AI requested a review from 304NotModified August 16, 2025 22:36
Copy link
Member

@304NotModified 304NotModified left a comment

Choose a reason for hiding this comment

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

@copilot see comments. For the links, make readable display links and make it consistent

- **Step Definition Injection**: Automatic dependency injection for step definition classes
- **Hook Injection**: Dependency injection support for before/after hooks
- **Lifecycle Management**: Proper scoping and lifecycle management of dependencies
- **Easy Configuration**: Simple setup and configuration
Copy link
Member

Choose a reason for hiding this comment

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

Remove this subjective promo line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the subjective promotional line "Easy Configuration" from the Autofac README. Changes committed in 3eb6338.


Install this package to add Autofac dependency injection support to your Reqnroll project:

```bash
Copy link
Member

Choose a reason for hiding this comment

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

Change to powershell (also others)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed all code blocks from bash to powershell syntax across all README files. Changes committed in 3eb6338.

Copy link
Member

Choose a reason for hiding this comment

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

Just name the package readmes, Readme.md

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Renamed all PackageReadme.md files to README.md across all packages. Updated corresponding project files and nuspec files to reference the new filenames. Changes committed in 3eb6338.

<NuspecFile>$(MSBuildThisFileDirectory)Reqnroll.Autofac.nuspec</NuspecFile>

<RootNamespace>Reqnroll.Autofac</RootNamespace>
<GenerateDocumentationFile>true</GenerateDocumentationFile>
Copy link
Member

Choose a reason for hiding this comment

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

Update the nuspec files for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated all nuspec files to reference README.md instead of PackageReadme.md in the files section. Changes committed in 3eb6338.

@@ -0,0 +1,29 @@
Reqnroll.SpecFlowCompatibility (pronounced as [reknroʊl]) provides compatibility layer for migrating from SpecFlow to Reqnroll.
Copy link
Member

Choose a reason for hiding this comment

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

Remove this (pronounced as [reknroʊl]) for all packages except reqnroll

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the pronunciation guide "(pronounced as [reknroʊl])" from all packages except the main Reqnroll package. Changes committed in 3eb6338.

@@ -0,0 +1,35 @@
Reqnroll.TUnit (pronounced as [reknroʊl]) enables using Reqnroll with TUnit test framework.

This package provides integration between Reqnroll and TUnit, allowing you to write BDD tests using Gherkin syntax that run with the TUnit test framework.
Copy link
Member

Choose a reason for hiding this comment

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

Add a link to https://tunit.dev/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added link to TUnit.dev with readable display text. Changes committed in 3eb6338.

@@ -0,0 +1,29 @@
Reqnroll.Verify (pronounced as [reknroʊl]) provides integration with Verify for snapshot testing in Reqnroll scenarios.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added link to Verify GitHub repository with readable display text. Changes committed in 3eb6338.

@@ -0,0 +1,29 @@
Reqnroll.Windsor (pronounced as [reknroʊl]) provides Castle Windsor dependency injection integration for Reqnroll.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added link to Castle Windsor GitHub repository with readable display text. Changes committed in 3eb6338.

@@ -0,0 +1,35 @@
Reqnroll.xUnit (pronounced as [reknroʊl]) enables using Reqnroll with xUnit test framework.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added link to xUnit.net with readable display text. Changes committed in 3eb6338.

@@ -0,0 +1,20 @@
Reqnroll.Generator (pronounced as [reknroʊl]) is the code generation component of the Reqnroll framework.

This package provides code generation capabilities that convert Gherkin feature files into executable test code for the Reqnroll BDD test automation framework for .NET.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added link to the generator configuration documentation with readable display text. Changes committed in 3eb6338.

…pdate links, change bash to powershell

Co-authored-by: 304NotModified <5808377+304NotModified@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Aug 21, 2025

@copilot see comments. For the links, make readable display links and make it consistent

Made all links readable with consistent display text formatting. All external documentation links now show meaningful text instead of raw URLs, and documentation links consistently use "Reqnroll Documentation" format. Changes committed in 3eb6338.

@304NotModified
Copy link
Member

superseded by #844

@gasparnagy gasparnagy deleted the copilot/fix-838c609e-2773-4068-b012-87d60b4350bd branch September 18, 2025 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants