Skip to content

Conversation

@rymdbar
Copy link

@rymdbar rymdbar commented Nov 9, 2025

This commit adds configuration files for the most common CMake linters, setting them up to ignore some of their default rules. I could see how changing the title to specifically refer to CMake could be appropriate. I have not made any attempts at working with e.g. the C++ files. Maybe additional setup is required to prevent ones editor to light up like a Christmas tree?

An alternative to merging this would of course be to adapt this project's style to follow best practices. In fact some lints ignored in this commit are related to indenting, which the existing open pull request #70 attempts to improve.

Ignoring the issue, thus requiring each and every potentially interested coder to maintain their own exceptions for the project, is seldom the greatest approach for attracting contributions.

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