You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by "Adam Hunyadi (Jira)" <ji...@apache.org> on 2020/06/26 12:16:00 UTC

[jira] [Created] (MINIFICPP-1271) Disable linter check for [build/include_subdir]

Adam Hunyadi created MINIFICPP-1271:
---------------------------------------

             Summary: Disable linter check for [build/include_subdir]
                 Key: MINIFICPP-1271
                 URL: https://issues.apache.org/jira/browse/MINIFICPP-1271
             Project: Apache NiFi MiNiFi C++
          Issue Type: Task
    Affects Versions: 0.7.0
            Reporter: Adam Hunyadi
            Assignee: Adam Hunyadi
             Fix For: 1.0.0


*Background:*

Including some of our dependencies produce linter errors, both by having them included int ""-s and <>-s. For example:
{code:c++}
#include "concurrentqueue.h"
{code}
{quote}error cpplint Include the directory when naming .h files [build/include_subdir] [4]
{quote}
{code:c++}
#include <concurrentqueue.h>
{code}
{quote}error cpplint Found C system header after C++ system header. Should be: ExampleHeader.h, c system, c++ system, other. [build/include_order] [4]
{quote}
The issue is that the google styleguide expects all dependency headers included in subfolders, while our project does not have this dependency.

*Proposal:*

As we do not gain much by enforcing a subfolder rule for every header file included, we agreed to disable this linter rule globally on the project.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)