microsoft / microsoft/component-detection

Use `sealed` where possible

Open
#456 0 comments 0 reactions 0 assignees View on GitHub
status:ready type:refactor
Dominant language
C#
Stars
553
Forks
135
Avg merge
20h 58m
Merged PRs (30d)
6

Description

In .NET 6, there were many performance improvements made to marking classes as `sealed`. [Microbenchmarks show somewhere between 2x to 500x performance improvement][1].

This may require #455 to be completed first.

[1]: https://devblogs.microsoft.com/dotnet/performance-improvements-in-net-6/#peanut-butter

Contributor guide

Open the contributing guide

Research direction

The issue does not name any files, tests, or entry points; first check the status of prerequisite issue #455 and identify the classes in component-detection that can be sealed. Use the linked .NET 6 performance guidance for context. Done means eligible classes are marked sealed without breaking the project's existing validation.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
performance, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.