cake-contrib / cake-contrib/Cake.Recipe

Use Typed Context for all Tasks

Open
#563 5 comments 0 reactions 0 assignees View on GitHub
Breaking change
Dominant language
C#
Stars
70
Forks
51
PR merge metrics
No merged PRs in 30d

Description

Currently, Cake.Recipe makes use of a static BuildParameters class to hold of the build time information.

Cake now has the concept of Typed Contexts, which makes it possible to create an instance of a class, which all tasks have access to, rather than having a static class instance.

Cake.Recipe should switch to using this where possible. It already has a partial implementation of this, but it would be good to completely remove the BuildParameters class.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the static BuildParameters class and the existing partial Typed Context implementation in Cake.Recipe. Trace how build-time information is accessed across the tasks, then verify that the remaining BuildParameters references have been removed and the tasks still receive the required context.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
build-system
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.