You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Timothy Chen (JIRA)" <ji...@apache.org> on 2014/11/12 00:22:34 UTC
[jira] [Closed] (MESOS-1291) Use clang-format to automatically
format code to style
[ https://issues.apache.org/jira/browse/MESOS-1291?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Timothy Chen closed MESOS-1291.
-------------------------------
commit 6404a401ce09f36c9e072c0d17ca15c278b0f9b5
Author: Michael Park <mc...@gmail.com>
Date: Tue Nov 11 14:30:09 2014 -0800
Introduced ClangFormat to Mesos.
Review: https://reviews.apache.org/r/26069
> Use clang-format to automatically format code to style
> ------------------------------------------------------
>
> Key: MESOS-1291
> URL: https://issues.apache.org/jira/browse/MESOS-1291
> Project: Mesos
> Issue Type: Improvement
> Components: technical debt
> Reporter: Dominic Hamon
> Assignee: Michael Park
> Labels: style
>
> Instead of relying on a script to check and report style errors, we should move to a workflow that allows people to write code how they feel comfortable and then automatically format it to conform to our style guide.
> The Chromium style from clang-format (http://clang.llvm.org/docs/ClangFormat.html) is very close to our style except for the dropped braces on class, struct, and function definitions, and two lines of whitespace between method definitions outside a class. As such, we should consider adopting clang-format and patching it to include a Mesos style variant.
> It can be run as part of post-reviews or as a git commit hook, or manually from within editors.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)