You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2021/07/29 14:39:08 UTC

[GitHub] [drill] dzamo opened a new pull request #2281: [MINOR UPDATE]: Add an .editorconfig file providing style hints to editors and IDEs

dzamo opened a new pull request #2281:
URL: https://github.com/apache/drill/pull/2281


   ## Description
   
   Non-code change.
   
   EditorConfig (https://editorconfig.org/) is an editor-neutral file format
   that holds code style settings.  This particular config has been defined
   to reflect the style convetions in the Drill codebase (most significantly,
   indentation is with 2 spaces).  Many mainstream editors and IDEs will
   recognise this file and set themselves up accordingly, making it easier to
   contribute code following the project's conventions.  Editors that do not
   understand EditorConfig are unaffected.
   
   Note that .editorconfig only contains global conventions for the codebase, not
   individual users' editor preferences, which must be masked by .gitignore.
   Other Apache projects with a .editorconfig file in their code repo are
   Superset, Subversion, Thrift, Submarine and presumably some more.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] luocooong commented on pull request #2281: [MINOR UPDATE]: Add an .editorconfig file providing style hints to editors and IDEs

Posted by GitBox <gi...@apache.org>.
luocooong commented on pull request #2281:
URL: https://github.com/apache/drill/pull/2281#issuecomment-889246337


   @dzamo 
   Error: Failed to execute goal org.apache.rat:apache-rat-plugin:0.13:check (rat-checks) on project drill-root: Too many files with unapproved license: 1 See RAT report in: /home/runner/work/drill/drill/target/rat.txt -> [Help 1]


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] dzamo commented on pull request #2281: [MINOR UPDATE]: Add an .editorconfig file providing style hints to editors and IDEs

Posted by GitBox <gi...@apache.org>.
dzamo commented on pull request #2281:
URL: https://github.com/apache/drill/pull/2281#issuecomment-889696045


   Thanks, I've added the Apache license.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] dzamo merged pull request #2281: [MINOR UPDATE]: Add an .editorconfig file providing style hints to editors and IDEs

Posted by GitBox <gi...@apache.org>.
dzamo merged pull request #2281:
URL: https://github.com/apache/drill/pull/2281


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org