Skip to content

[Repo Assist] Add doc comments to CsvExtensions, TextConversions, and HtmlActivePatterns (batch 1 of #1678)#1681

Merged
dsyme merged 2 commits intomainfrom
repo-assist/docs-issue-1678-batch1-363f88a263e78cf2
Mar 1, 2026
Merged

[Repo Assist] Add doc comments to CsvExtensions, TextConversions, and HtmlActivePatterns (batch 1 of #1678)#1681
dsyme merged 2 commits intomainfrom
repo-assist/docs-issue-1678-batch1-363f88a263e78cf2

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 1, 2026

🤖 This PR was created by Repo Assist, an automated AI assistant.

Part of #1678 — adds XML doc comments to three files that were missing documentation.

What's changed

src/FSharp.Data.Csv.Core/CsvExtensions.fs

Added <summary>, (param) doc comments to all 9 StringExtensions extension methods (AsInteger, AsInteger64, AsDecimal, AsFloat, AsBoolean, AsDateTime, AsDateTimeOffset, AsTimeSpan, AsGuid).

src/FSharp.Data.Runtime.Utilities/TextConversions.fs

Added <summary> and (param) doc comments to all TextConversions static methods:

  • AsInteger, AsInteger64, AsDecimal, AsFloat, AsBoolean
  • AsDateTime, AsDateTimeOffset, AsTimeSpan
  • AsDateOnly, AsTimeOnly (NET6+ methods)
  • AsGuid

src/FSharp.Data.Html.Core/HtmlActivePatterns.fs

Added doc comment to the module and to both active patterns:

  • (|HtmlElement|HtmlText|HtmlComment|HtmlCData|) — decomposes HtmlNode
  • (|HtmlAttribute|) — decomposes HtmlAttribute

Test Status

Generated by Repo Assist for issue #1678

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@da02dddebe71d9a937665abdcd1f5214dab852a7

…terns

Documents public string conversion extension methods in StringExtensions,
all TextConversions static methods, and the HtmlActivePatterns active patterns.

Part of #1678.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor Author

github-actions bot commented Mar 1, 2026

🤖 This is an automated note from Repo Assist.

⚠️ Important: This PR's description contains Fixes #1678, which will automatically close issue #1678 when merged. However, the maintainer (@dsyme) explicitly asked that #1678 remain open until the documentation work is complete (see the issue body).

Please remove the Fixes #1678 line from the PR description before merging, or change it to a reference like Part of #1678 to avoid auto-closing the tracking issue.

Sorry for the oversight!

Generated by Repo Assist

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@da02dddebe71d9a937665abdcd1f5214dab852a7

@dsyme dsyme marked this pull request as ready for review March 1, 2026 13:35
@dsyme dsyme merged commit f893f0c into main Mar 1, 2026
2 checks passed
@dsyme dsyme deleted the repo-assist/docs-issue-1678-batch1-363f88a263e78cf2 branch March 1, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document all code in this codebase

1 participant