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.

Sign up for GitHub

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

Jump to bottom

Documentation should reflect new clang-format enforced style #1270

Closed
dzenanz opened this issue Sep 25, 2019 · 6 comments
Closed

Documentation should reflect new clang-format enforced style #1270

dzenanz opened this issue Sep 25, 2019 · 6 comments
Assignees
@hjmjohnson
Labels
status:Backlog Postponed without a fixed deadline type:Documentation Documentation improvement or change

Comments

@dzenanz
Copy link
Member

dzenanz commented Sep 25, 2019

#1191 changed the code style, documentation should reflect this change, both style-wise and process-wise.
@dzenanz dzenanz added the type:Documentation Documentation improvement or change label Sep 25, 2019
@hjmjohnson
Copy link
Member 

ITKSoftwareGuide/SoftwareGuide/Latex/Appendices/CodingStyleGuide.tex

@hjmjohnson hjmjohnson self-assigned this Sep 25, 2019
@jhlegarreta
Copy link
Member

Isn't this issue more appropriate for the ITK SW Guide?
InsightSoftwareConsortium/ITKSoftwareGuide#133

@dzenanz
Copy link
Member Author

dzenanz commented Sep 27, 2019

I forgot we have a separate repository for documentation, but there is documentation outside of software guide, e.g. https://github.com/InsightSoftwareConsortium/ITK/blob/master/Documentation/GitHelp.md.

@jhlegarreta
Copy link
Member

True, but that folder does not contain any style guide-related contents if I am not mistaken, right?

@dzenanz
Copy link
Member Author

dzenanz commented Sep 27, 2019

There is now a git hook which is being run by default, I believe.

@stale
Copy link

stale bot commented Jan 25, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the status:Backlog Postponed without a fixed deadline label Jan 25, 2020
@dzenanz dzenanz closed this as completed Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hjmjohnson hjmjohnson

Labels
status:Backlog Postponed without a fixed deadline type:Documentation Documentation improvement or change
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@hjmjohnson @dzenanz @jhlegarreta