dev: Document astyle python script #6292
Conversation
Signed-off-by: Ryan Friedman <[email protected]>
| @@ -21,8 +21,15 @@ automatically according to the guidelines below. | |||
|
|
|||
| astyle --options=Tools/CodeStyle/astylerc <filename> | |||
|
|
|||
| Some files are enforced to be *astyle* clean. To format them automatically, run: | |||
There was a problem hiding this comment.
not sure we want to push users to run this all the time when only a few files are affected,,,,maybe we should specify that it should be checked if you have modified the following ..... libraries or files?
There was a problem hiding this comment.
More maintenance burden, @Hwurzburg ?
We should probably put this advice - run astyle - into the CI error message when one of these files fails the checks, if we haven't already. It's much more likely to be useful there.
There was a problem hiding this comment.
I don't think the existing recommendation is useful because it
- Doesn't tell you which file to use
- None of the existing code is enforced to the style guide, so it would cause unrelated changes
Proposing a reformat of the codebase was rejected in dev call because it would make existing PR's and backports much more difficult to rebase. Thus, I would propose recommending making new files astyle clean, and then we document that in the wiki.
There was a problem hiding this comment.
what
* Doesn't tell you which file to use
you mean ? if you think that is an issue we could clarify what refers to
* None of the existing code is enforced to the style guide, so it would cause unrelated changes
its true when run on the file you have modified it would correct existing format issues....maybe we could start allowing an unsquashed ending commit in PRs for style changes to allow any new touched file to be astyle corrected?
Document the astyle format script, and how we can ignore format commits in the git blame.