microsoft / microsoft/microsoft-performance-toolkit-sdk
Rearchitect such that the Engine knows less of the Runtime
Open
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 239
- Forks
- 78
- Avg merge
- 4d 3h
- Merged PRs (30d)
- 2
Description
Too many classes are public in the runtime right now for the Engine. We'd like to move some of the Engine code behind a new Runtime class to further hide runtime implementation from the Engine.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by mapping the Engine's dependencies on the currently public runtime classes and review how a new Runtime class could own that Engine-facing work. Define the boundary between Engine and Runtime before changing code. Done means the Engine relies on the new Runtime abstraction and the relevant runtime implementation details are no longer public.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100