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

New Linter: clang-format #507

Closed
manicmaniac opened this issue Aug 20, 2022 · 3 comments · Fixed by #519
Closed

New Linter: clang-format #507

manicmaniac opened this issue Aug 20, 2022 · 3 comments · Fixed by #519
Labels
enhancement New feature or request good first issue Good for newcomers in progress

Comments

@manicmaniac
Copy link
Contributor

It would be great if lint-action adds support for clang-format.
clang-format is de facto standard for C / C++ / Objective-C families and it provides auto fix feature.

@ocean90 ocean90 added enhancement New feature or request good first issue Good for newcomers labels Aug 29, 2022
@ocean90
Copy link
Member

ocean90 commented Aug 29, 2022

Thanks for the request, sounds good to me. Are you able to add the linter by yourself?

Take a look at https://github.com/wearerequired/lint-action/blob/master/CONTRIBUTING.md#adding-a-new-linter to see what's needed to get a new linter/formatter added.

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Sep 14, 2022
@manicmaniac
Copy link
Contributor Author

I'm trying to implement clang-format linter now 👍

@github-actions github-actions bot removed the stale label Sep 15, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers in progress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants