Skip to content

Conversation

@TimothyMakkison
Copy link
Contributor

@TimothyMakkison TimothyMakkison commented Jan 8, 2026

Fix indentation and remove blank line. Working on a refactor in file and noticed the issue.

I would suggest something like csharpier but it's pretty different to the existing code style

@thomhurst
Copy link
Owner

Summary

Fixes indentation for BuildBeforeTestHooksAsync and BuildAfterTestHooksAsync methods and removes trailing blank line.

Critical Issues

None found ✅

Verdict

APPROVE - No critical issues

This is a pure formatting change with no functional impact. The indentation now matches the rest of the methods in the file.

@TimothyMakkison TimothyMakkison force-pushed the reformat_hook_collection branch from c14be21 to 54bf259 Compare January 8, 2026 16:39
@thomhurst thomhurst merged commit 0a8a649 into thomhurst:main Jan 9, 2026
9 of 10 checks passed
This was referenced Jan 12, 2026
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