cake-build / cake-build/cake

Inject line processors in ScriptAnalyzer

Open
#1,358 0 comments 2 reactions 0 assignees View on GitHub
Improvement
Dominant language
C#
Stars
4.2k
Forks
778
Avg merge
3h 37m
Merged PRs (30d)
21

Description

At the moment, line processors are created within the [ScriptAnalyzer](https://github.com/cake-build/cake/blob/develop/src/Cake.Core/Scripting/Analysis/ScriptAnalyzer.cs). Those should be injected instead. We should also add tests that make sure that the [CoreModule](https://github.com/cake-build/cake/blob/develop/src/Cake.Core/Modules/CoreModule.cs) registers all processors.

This will make it possible to add custom line processors via modules.

Contributor guide

Open the contributing guide

Research direction

Start by reading src/Cake.Core/Scripting/Analysis/ScriptAnalyzer.cs to locate how line processors are created, then inspect src/Cake.Core/Modules/CoreModule.cs. Add coverage confirming CoreModule registers all processors and verify that processors can be supplied through modules; run the relevant test suite to confirm the registrations and analysis still work.

Written by the indexing model from the issue text.

Assessment

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.