Skip to content

Conversation

@dorssel
Copy link
Owner

@dorssel dorssel commented Nov 14, 2025

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 14, 2025

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 8 0 0 0.12s
✅ C cppcheck 2 0 0 0.09s
✅ C cpplint 2 0 0 0.32s
✅ CSS stylelint 1 0 0 1.45s
✅ EDITORCONFIG editorconfig-checker 152 0 0 0.76s
✅ HTML djlint 2 0 0 1.68s
✅ HTML htmlhint 2 0 0 0.32s
✅ JSON jsonlint 4 0 0 0.17s
✅ JSON prettier 4 0 0 0.76s
✅ MARKDOWN markdownlint 2 0 0 0.73s
✅ MARKDOWN markdown-table-formatter 2 0 0 0.47s
✅ REPOSITORY gitleaks yes no no 1.24s
✅ REPOSITORY git_diff yes no no 0.02s
✅ REPOSITORY grype yes no no 32.05s
✅ REPOSITORY secretlint yes no no 1.49s
✅ REPOSITORY syft yes no no 2.88s
✅ REPOSITORY trivy-sbom yes no no 0.9s
✅ REPOSITORY trufflehog yes no no 4.3s
✅ XML xmllint 16 0 0 1.01s
✅ YAML prettier 19 0 0 1.09s
✅ YAML v8r 19 0 0 11.35s
✅ YAML yamllint 19 0 0 0.95s

See detailed reports in MegaLinter artifacts

You could have the same capabilities but better runtime performances if you use a MegaLinter flavor:

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx [email protected] --custom-flavor-setup --custom-flavor-linters ACTION_ACTIONLINT,C_CPPCHECK,C_CPPLINT,CSS_STYLELINT,EDITORCONFIG_EDITORCONFIG_CHECKER,HTML_DJLINT,HTML_HTMLHINT,JSON_JSONLINT,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_GIT_DIFF,REPOSITORY_GITLEAKS,REPOSITORY_GRYPE,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,XML_XMLLINT,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is graciously provided by OX Security

@codecov
Copy link

codecov bot commented Nov 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (cac83f0) to head (9139533).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #206   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines          861       861           
  Branches        79        79           
=========================================
  Hits           861       861           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dorssel dorssel force-pushed the net10 branch 2 times, most recently from cb4d241 to bcbab06 Compare November 15, 2025 20:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates the project from .NET 9 to .NET 10 SDK, updating dependencies and configurations to support the new SDK version. The changes include modifications to WASM build processes and DllImport search path handling.

Key changes:

  • Updated SDK version from 9.0.306 to 10.0.100 with corresponding framework and language version updates
  • Consolidated DllImport search paths from individual method attributes to a single assembly-level attribute
  • Modified WASM build process to handle new .NET 10 requirements for object file naming

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
global.json Updated SDK to 10.0.100, MSTest.Sdk to 4.0.2, and added test runner configuration
Directory.Build.props Changed target framework to net10.0 and C# language version to 14.0
Directory.Packages.props Updated ASP.NET Core WebAssembly packages to 10.0.0 and FluentUI packages to 4.13.1
Xmss/AssemblySettings.cs Added assembly-level DefaultDllImportSearchPaths attribute
Xmss/GlobalSuppressions.cs Removed suppression for CA5393 (unsafe DllImportSearchPath)
Xmss/Internal/*.cs Removed method-level DefaultDllImportSearchPaths attributes from all LibraryImport declarations
libc/xmss.c Added wrapper file to include libxmss.c for WASM compatibility
libc/wasm.diff Added patch file to change CMakeLists.txt for WASM builds
.github/workflows/runtime-browser-wasm.yml Updated build process to apply WASM patch and use new artifact naming
.github/workflows/build.yml Updated test command syntax for new Microsoft.Testing.Platform
Examples/WebAssembly/App.razor Removed NotFound route handling section
.editorconfig Added C# style preferences and diff/patch file configuration
.gitattributes Added .diff file line ending configuration
.config/dotnet-tools.json Updated gitversion.tool to 6.5.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dorssel dorssel marked this pull request as ready for review November 15, 2025 20:36
@dorssel dorssel merged commit 04cbd44 into main Nov 15, 2025
23 checks passed
@dorssel dorssel deleted the net10 branch November 15, 2025 20:37
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