FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore: Convert to SLNX, move test to ./src/tests folder, add claude instrutions by glennawatson · Pull Request #1457 · reactiveui/splat · GitHub

/ splat Public

chore: Convert to SLNX, move test to ./src/tests folder, add claude instrutions - #1457

Merged
glennawatson merged 1 commit into
mainfrom
glennawatson/tests-folder
Dec 30, 2025
ConversationCommits (1)ChecksFiles changed
Merged

chore: Convert to SLNX, move test to ./src/tests folder, add claude instrutions#1457
glennawatson merged 1 commit into
mainfrom
glennawatson/tests-folder

Conversation

glennawatson commented Dec 30, 2025

Copy link
Copy Markdown
Contributor
  • Move tests to ./src/tests
  • add claude instructions
  • use slnx instead of sln
  • Make IL analyzers default to error.

Analyzer configuration enhancements:

  • Added error severity for all .NET trimming analyzer warnings (IL2001–IL2123), which cover issues like missing or incorrect RequiresUnreferencedCode and DynamicallyAccessedMembers attributes, unresolved dependencies, and reflection patterns that may break after trimming.
  • Added error severity for .NET AOT analyzer warnings (IL3050–IL3057), which address problems with dynamic code requirements in native AOT scenarios, such as improper use of RequiresDynamicCode attributes and unsupported generic expansions.

codecov Bot commented Dec 30, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.58%. Comparing base (970926e) to head (1cb73fe).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1457   +/-   ##
=======================================
  Coverage   41.58%   41.58%           
=======================================
  Files         147      147           
  Lines        7805     7805           
  Branches      805      805           
=======================================
  Hits         3246     3246           
  Misses       4391     4391           
  Partials      168      168

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

glennawatson merged commit 7172f63 into main Dec 30, 2025
6 checks passed
glennawatson deleted the glennawatson/tests-folder branch December 30, 2025 03:04

github-actions Bot commented Jan 14, 2026

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

github-actions Bot locked as resolved and limited conversation to collaborators Jan 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

ChrisPulman ChrisPulman approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL