cake-build / cake-build/cake

Add interface for FilePath, DirectoryPath and Path.

Open
#2,074 0 comments 4 reactions 0 assignees View on GitHub
Improvement
Dominant language
C#
Stars
4.2k
Forks
778
Avg merge
1h 15m
Merged PRs (30d)
19

Description

By implementing these interfaces on `ConvertableDirectoryPath` and `ConvertableFilePath` usage of paths received from the `Directory(..)` and `File(..)` aliases will be much easier.

We should also change all internal methods from taking `FilePath`/`DirectoryPath` to `IFilePath`/`IDirectoryPath`.

Another benefit this brings is that we remove A LOT of implicit conversions like ConvertableFilePath -> DirectoryPath which is always nice.

Contributor guide

Open the contributing guide

Research direction

Start by locating ConvertableDirectoryPath, ConvertableFilePath, FilePath, DirectoryPath, IFilePath, and IDirectoryPath, then trace the Directory(..) and File(..) aliases that produce these values. Done means the two convertable types implement the interfaces, internal methods accept the interface types, and the unnecessary implicit conversions are removed.

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
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.