baidu / baidu/tera

Rename IsExist() to PathExists()?

Open
#1,022 1 comment 1 reaction 0 assignees View on GitHub
class/enhancement module/ts stat/will-do-later
Dominant language
C++
Stars
1.9k
Forks
432
PR merge metrics
No merged PRs in 30d

Description

IsExist() does not really read good in English and it is probably grammatically incorrect.

Consider renaming this function to PathExists()?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by searching the repository for IsExist() to find its definition and every caller. Check whether tests or public API references also use the name; done means consistently applying the agreed PathExists() rename and confirming the project still builds and its tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
databases
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 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.