Skip to content

feat: add clang-format, clang-tidy and other clang-tools plugins - #1095

Open
shenxianpeng wants to merge 1 commit into
asdf-vm:masterfrom
shenxianpeng:add-clang-tools-plugin
Open

shenxianpeng wants to merge 1 commit into
asdf-vm:masterfrom
shenxianpeng:add-clang-tools-plugin

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented May 29, 2025 •

Copy link
Copy Markdown

Summary

Description:

asdf-clang-tools picks the tool to install from the plugin name, so it is registered once per tool, the same way as asdf-hashicorp:

  • clang-apply-replacements
  • clang-format
  • clang-include-cleaner
  • clang-query
  • clang-tidy

Checklist

  • Format with scripts/format.bash
  • Test locally with scripts/test_plugin.bash --file plugins/<your_new_plugin_name>
  • Your plugin CI tests are green
    • Tip: use the plugin_test action from asdf-actions in your plugin CI

@shenxianpeng

Copy link
Copy Markdown
Author

Hi @Stratus3D, sorry for the ping. Could you please take a look at this PR when you have a chance?

Happy to make any changes if needed. Thanks!

@shenxianpeng
shenxianpeng force-pushed the add-clang-tools-plugin branch from 1d81437 to 0ef7721 Compare October 1, 2026 07:18
asdf-clang-tools picks the tool to install from the plugin name, so
register it once per tool, like asdf-hashicorp: clang-apply-replacements,
clang-format, clang-include-cleaner, clang-query and clang-tidy.
@shenxianpeng
shenxianpeng force-pushed the add-clang-tools-plugin branch from 0ef7721 to ac40589 Compare October 1, 2026 07:39
@shenxianpeng shenxianpeng changed the title feat: Add clang-tools plugin feat: add clang-format, clang-tidy and other clang-tools plugins Oct 1, 2026
@shenxianpeng

Copy link
Copy Markdown
Author

Hi @Stratus3D, I've rebased this onto the latest master to resolve the conflict with the table re-format from #1089.

I also switched from a single clang-tools shortname to one per tool (clang-apply-replacements, clang-format, clang-include-cleaner, clang-query, clang-tidy). The plugin picks the tool from the plugin name, so a plugin added as clang-tools couldn't install anything. This follows the same pattern as asdf-hashicorp. scripts/test_plugin.bash --diff and prettier --check both pass locally.

Could you approve the workflow runs when you have a moment? Thanks!

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.

1 participant