Skip to content

Fix compliance findings in Frends.AzureBlobStorage.DownloadBlob (FT0013 ×4, FT0019 ×1) - #127

Merged
jefim merged 2 commits into
mainfrom
copilot/fix-compliance-findings-blob
Jul 24, 2026
Merged

Fix compliance findings in Frends.AzureBlobStorage.DownloadBlob (FT0013 ×4, FT0019 ×1)#127
jefim merged 2 commits into
mainfrom
copilot/fix-compliance-findings-blob

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Resolves 5 task analyzer findings in Frends.AzureBlobStorage.DownloadBlob: four unsupported <see cref> XML doc tags and one incorrect parameter tab order.

Changes

FT0013 — Remove unsupported <see cref> tags

Replaced <see cref="..."/> with plain text in four doc comments:

  • Error.cs — class summary
  • Options.csOtherEncoding property
  • Result.csFilePath and Error properties

FT0019 — Fix parameter order

Reordered DownloadBlob signature to match the required standard: Input → Connection → Options → CancellationToken.

Before:

public static async Task<Result> DownloadBlob(
    [PropertyTab] Input input, [PropertyTab] Options options,
    [PropertyTab] Connection connection, CancellationToken cancellationToken)

After:

public static async Task<Result> DownloadBlob(
    [PropertyTab] Input input, [PropertyTab] Connection connection,
    [PropertyTab] Options options, CancellationToken cancellationToken)

This is a breaking change — the Connection and Options tabs swap positions in the Frends UI. Test calls updated accordingly.

Version

3.0.04.0.0 (major bump for breaking signature change)

Copilot AI changed the title [WIP] Fix compliance findings in Frends.AzureBlobStorage.DownloadBlob Fix compliance findings in Frends.AzureBlobStorage.DownloadBlob (FT0013 ×4, FT0019 ×1) Jul 23, 2026
Copilot finished work on behalf of jefim July 23, 2026 05:31
Copilot AI requested a review from jefim July 23, 2026 05:31
@jefim
jefim marked this pull request as ready for review July 24, 2026 04:48
@jefim
jefim merged commit 38f0318 into main Jul 24, 2026
3 checks passed
@jefim
jefim deleted the copilot/fix-compliance-findings-blob branch July 24, 2026 04:55
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