Add .NET 10 reference skill and install relevant dotnet/skills into repo#530
Merged
Conversation
Add a repo-committed Copilot skill at .github/skills/dotnet10-features that summarizes the .NET 10 (and bundled C# 14) features relevant to this repository, mapping each feature to concrete ReactiveProperty scenarios (field keyword, extension members, null-conditional assignment, Blazor 10 form validation, etc.) and documenting the LangVersion/TFM constraints. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Install 14 ReactiveProperty-relevant agent skills from dotnet/skills@v1.0.0 at project scope via 'gh skill install', so contributors who clone the repo get them (instead of a per-user plugin/marketplace install). Skills (MSBuild/multi-targeting, MSTest testing, .NET upgrade, benchmarking, NuGet publishing) are placed under .agents/skills with gh-injected source tracking metadata for 'gh skill update'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Follow-up to the initial skill install. Adds Tier A+B skills relevant to ReactiveProperty: - dotnet-test/dotnet-test-frameworks and dotnet-test/filter-syntax: reference data that the already-installed test-anti-patterns and run-tests skills load - dotnet-upgrade/dotnet-aot-compat: trim/AOT compatibility for the library (relevant for Blazor wasm trimming and AOT consumers) - dotnet-test/migrate-vstest-to-mtp and migrate-mstest-v3-to-v4: test runner and MSTest version modernization - dotnet-diag/analyzing-dotnet-performance: perf anti-pattern auditing to complement microbenchmarking - dotnet-msbuild/binlog-failure-analysis and incremental-build: multi-target build failure diagnosis and incremental-build correctness All pinned to dotnet/skills@v1.0.0 at project scope under .agents/skills. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
ReactiveProperty already multi-targets up to
net10.0(the Blazor package and samples) while the core libraries stay on C# 12. This PR makes the repo's agent-assisted workflows aware of that context by shipping curated skills directly in the repository, so anyone who clones the repo gets them without a per-user plugin/marketplace install.What changed
1. New repo-authored skill:
.github/skills/dotnet10-features/SKILL.mdA curated, English reference that maps the .NET 10 / C# 14 features that actually matter to this codebase, with before/after examples grounded in ReactiveProperty:
fieldkeyword and extension members (highest relevance: INPC property accessors and the heavyReactive.Bindingsextension-method surface)nameofon unbound generics, user-defined compound assignmentReactiveProperty.Platform.Blazor(improved form validation,OwningComponentBaseasync disposal,[PersistentState])LangVersion 12and do not targetnet10.0; runtime/BCL-gated features must be#if NET10_0_OR_GREATER-guarded across thenetstandard2.0/net472legs2. Installed 14 relevant skills from dotnet/skills into
.agents/skills/Installed at project scope via
gh skill install(pinned to tagv1.0.0), selected for what this repo actually is and does:directory-build-organization,msbuild-modernization,msbuild-antipatterns,check-bin-obj-clash,resolve-project-referenceswriting-mstest-tests,run-tests,test-anti-patterns,coverage-analysismigrate-dotnet8-to-dotnet9,migrate-dotnet9-to-dotnet10,migrate-nullable-referencesBenchmark/projects (1):microbenchmarkingnuget-trusted-publishingEach installed skill carries gh-injected source-tracking frontmatter (
github-repo,github-path,github-pinned: v1.0.0) sogh skill updatecan refresh them later.Notes for reviewers
.github/skills/(Copilot's repo skill location), whilegh skillinstalls the shared, multi-agent skills under.agents/skills/(the agentskills.io standard location shared by Copilot, Cursor, Codex, etc.).v1.0.0, which does not yet ship thedotnet-blazorordotnet11plugins that appear onmain; those can be added once tagged.