Skip to content

chore: CodeMaid cleanup of library and test sources - #132

Merged
AtwoodTM merged 2 commits into
masterfrom
chore/codemaid-cleanup
Sep 26, 2026
Merged

AtwoodTM merged 2 commits into
masterfrom
chore/codemaid-cleanup

Conversation

@AtwoodTM

Copy link
Copy Markdown
Collaborator

CodeMaid was run across the library and the tests. The diff is trailing whitespace and extra blank lines. 17 files, +130 / −151 against e0af88c. Ignoring whitespace, what remains is blank-line removal in six files: Dyn.fs, Operators.fs, SymbolicString.fs, TopLevelOperators.fs, Tests/Library1.fs, and Tests/Raw.fs.

No identifiers, signatures, or behavior change. dotnet test should still pass.

Closes #129.

Trailing whitespace and extra blank lines only. No signature or behavior change.
Copilot AI lite review requested due to automatic review settings September 26, 2026 15:50
@AtwoodTM
AtwoodTM requested a review from jbtule as a code owner September 26, 2026 15:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved issues; changes are limited to formatting cleanup.

Review effort: Lite
Findings: None

What changed in this PR

Formatting-only CodeMaid cleanup across 17 F# library and test files, preserving behavior and APIs.

Changes:

  • Removed trailing whitespace.
  • Removed redundant blank lines.
  • Preserved identifiers, signatures, and test behavior.
File Reviewed changes
Tests/​WhyExamples.fs Whitespace cleanup
Tests/​TryGet.fs Whitespace cleanup
Tests/​ReadmeExamples.fs Whitespace cleanup
Tests/​Raw.fs Whitespace and blank-line cleanup
Tests/​Quotations.fs Whitespace cleanup
Tests/​Properties.fs Whitespace cleanup
Tests/​OperatorValues.fs Whitespace cleanup
Tests/​Obsolete.fs Whitespace cleanup
Tests/​Library1.fs Whitespace and blank-line cleanup
Tests/​Invocation.fs Whitespace cleanup
FSharp.Interop.Dynamic/​TopLevelOperators.fsi Signature formatting cleanup
FSharp.Interop.Dynamic/​TopLevelOperators.fs Whitespace cleanup
FSharp.Interop.Dynamic/​SymbolicString.fs Whitespace cleanup
FSharp.Interop.Dynamic/​Operators.fsi Signature formatting cleanup
FSharp.Interop.Dynamic/​Operators.fs Whitespace cleanup
FSharp.Interop.Dynamic/​Dyn.fsi Signature formatting cleanup
FSharp.Interop.Dynamic/​Dyn.fs Whitespace cleanup

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@AtwoodTM
AtwoodTM merged commit c81f418 into master Sep 26, 2026
18 checks passed
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.

CodeMaid cleanup of the library and test sources

2 participants