Skip to content

[infra] Update pre-commit config to exclude yapf#16316

Merged
hseok-oh merged 1 commit intoSamsung:masterfrom
hseok-oh:fix_yapf_bug
Nov 27, 2025
Merged

[infra] Update pre-commit config to exclude yapf#16316
hseok-oh merged 1 commit intoSamsung:masterfrom
hseok-oh:fix_yapf_bug

Conversation

@hseok-oh
Copy link
Copy Markdown
Contributor

This commit updates yapf exclude to add files in .yapfignore list.
It is a workaround to avoid errors when pre-commit yapf tries to check file in .yapfignore list.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh hseok82.oh@samsung.com


Related issue: https://stackoverflow.com/questions/68274102/pre-commit-yapf-fails-on-file-included-in-yapfignore

This commit updates yapf exclude to add files in .yapfignore list.
It is a workaround to avoid errors when pre-commit yapf tries to check file in .yapfignore list.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Nov 26, 2025
@hseok-oh hseok-oh merged commit 0c60161 into Samsung:master Nov 27, 2025
6 checks passed
@hseok-oh hseok-oh deleted the fix_yapf_bug branch November 27, 2025 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR/ready for review It is ready to review. Please review it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants