Skip to content

Conversation

@OsirisTerje
Copy link
Member

Moved examples to the snippets
Added info on StopOnFailure property
Added warning on NUnit Issue 5031

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates documentation for the RepeatAttribute by adding examples, clarifying behavior options, and noting a known issue. The documentation now better explains the StopOnFailure property and includes practical code examples.

  • Moved inline code examples to dedicated snippet files for better maintainability
  • Added documentation for the StopOnFailure property available from version 4.3.0
  • Added warning about NUnit issue 5031 affecting console output and failure reporting

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/snippets/Snippets.NUnit/Attributes/RepeatAttributeExample.cs New snippet file containing three test examples demonstrating different RepeatAttribute behaviors
docs/articles/nunit/writing-tests/attributes/repeat.md Updated documentation with improved structure, examples references, and additional behavior details

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@SeanKilleen SeanKilleen left a comment

Choose a reason for hiding this comment

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

Generally looks good to me! Just one question/suggestion.

@OsirisTerje OsirisTerje merged commit 473bcc4 into master Sep 14, 2025
7 checks passed
@OsirisTerje OsirisTerje deleted the repeatAttribute branch September 14, 2025 21:23
github-actions bot pushed a commit that referenced this pull request Sep 14, 2025
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.

3 participants