beyond-all-reason / beyond-all-reason/RecoilEngine

Consider `VFS.RecursiveDirList`

Open
#1,487 0 comments 0 reactions 0 assignees View on GitHub
area: Lua API good first issue
Dominant language
C++
Stars
683
Forks
293
Avg merge
3d 2h
Merged PRs (30d)
40

Description

`VFS.RecursiveDirList(path, filter, mode)` would be equivalent to current `VFS.DirList(path, filter, mode, true)` since the 4th bool arg is whether the search is recursive. Named functions are clearer (for both reading and writing) than bool args.

Contributor guide

Open the contributing guide

Research direction

Start by locating the declaration and uses of VFS.DirList, especially calls that pass true as the recursive argument. Check how the VFS API is exposed and tested, then determine whether the named RecursiveDirList equivalent can be added without changing existing behavior. Done means the clearer named entry point is available and its recursive behavior is covered or verified.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend-api-design
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.