Skip to content

fix: resolve build errors and warnings#31

Merged
phmatray merged 1 commit intomainfrom
fix/build-errors-warnings
Feb 28, 2026
Merged

fix: resolve build errors and warnings#31
phmatray merged 1 commit intomainfrom
fix/build-errors-warnings

Conversation

@phmatray
Copy link
Contributor

Summary

  • Fix 2 build errors: Replace removed SendResultAsync with IResult.ExecuteAsync(HttpContext) (FastEndpoints v8 API change)
  • Fix 27 analyzer warnings: file headers, static methods, generic overloads, discards, keyword conflicts
  • Update .editorconfig: correct file_header_template from TaLibStandard to FastComponents

Errors Fixed

Code Issue Fix
CS0103 SendResultAsync does not exist Use response.ExecuteAsync(HttpContext)

Warnings Fixed

Code Count Fix
IDE0073 13 Update .editorconfig header + add headers to all files
CA1716 2 Suppress for As property (breaking change to rename)
CA1822 2 Make HtmlBeautifier.BeautifyHtml static
CA2263 2 Use Activator.CreateInstance<T>() generic overload
IDE0058 7 Add discards (_ =) for unused expressions
IDE0340 1 Use unbound generic type in nameof

Resolves failing run #22526226772

Test plan

  • CI passes on this PR (0 errors, 0 new warnings)
  • NuGet package builds successfully

Errors fixed:
- CS0103: Replace removed SendResultAsync with IResult.ExecuteAsync(HttpContext)
- Update ComponentHtmlResponseService to call static HtmlBeautifier.BeautifyHtml

Warnings fixed:
- IDE0073: Update .editorconfig file_header_template from TaLibStandard to FastComponents
  and add correct copyright headers to all 13 source files
- CA1716: Suppress for HtmxComponentBase.As (renaming would be a breaking change)
- CA1822: Make HtmlBeautifier.BeautifyHtml static
- CA2263: Use generic Activator.CreateInstance<T>() overload
- IDE0058: Add discards for unused expression values
- IDE0340: Use unbound generic type in nameof expression
@phmatray phmatray merged commit e693788 into main Feb 28, 2026
5 checks passed
@phmatray phmatray deleted the fix/build-errors-warnings branch February 28, 2026 18:22
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.

1 participant