Skip to content

Conversation

@k-yle
Copy link
Member

@k-yle k-yle commented Mar 10, 2025

If --stat is invoked without the path argument, then mergedConfig.stat becomes a boolean, and causes the program to crash here:

if (mergedConfig.stat) {
writeJSONSync(mergedConfig.stat, projectStat, { spaces: 2 });
}

@CLAassistant
Copy link

CLAassistant commented Mar 10, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@octogonz octogonz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @k-yle !

@octogonz octogonz merged commit 22d1374 into tiktok:main Mar 11, 2025
2 checks passed
@k-yle k-yle deleted the kh/stat branch March 11, 2025 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants