Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[clang-apply-replacements] Add support for the .yml file extension #78842

Conversation

unterumarmung
Copy link
Contributor

The .yml file extension is a valid extension for the YAML files, but it was not previously supported by the Clang Apply Replacements tool. This commit adds support for processing .yml files. Without this change, running the tool on a folder containing .yml files generated by clang-tidy would have no effect.

The `.yml` file extension is a valid extension for the YAML files, but it was not previously supported by the Clang Apply Replacements tool. This commit adds support for processing `.yml` files. Without this change, running the tool on a folder containing `.yml` files generated by clang-tidy would have no effect.
@unterumarmung unterumarmung force-pushed the clang-apply-replacements-add-yml-extension-support branch from 9c6d5fe to 14767cd Compare January 20, 2024 11:10
Copy link
Member

@PiotrZSL PiotrZSL left a comment

Choose a reason for hiding this comment

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

LGTM, you may want to wait like 1-2 day before merging so if someone else would have some comments.

@unterumarmung unterumarmung merged commit 45fec0c into llvm:main Jan 25, 2024
4 checks passed
harbandana pushed a commit to harbandana/LLVM-REPO that referenced this pull request Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants