The checks would be run as part of the build, and would fail for any style violations.
Discuss the conventions on the PR if you find particular changes that we should discuss. I included a summary of the common changes in the PR description – feel free to quote these and bring them up for discussion as well. -Matt Cheah From: Ryan Blue <rb...@netflix.com> Reply-To: "rb...@netflix.com" <rb...@netflix.com> Date: Friday, March 22, 2019 at 9:27 AM To: Matt Cheah <mch...@palantir.com> Cc: "dev@iceberg.apache.org" <dev@iceberg.apache.org> Subject: Re: Style guidelines proposal for Iceberg Thanks for working on this, Matt! If I understand correctly, the checks are run as part of the build, so Travis CI builds will fail for style violations? Where should we discuss the specific conventions? Would you prefer in this thread or on the PR? On Thu, Mar 21, 2019 at 5:07 PM Matt Cheah <mch...@palantir.com> wrote: Hi everyone, A prerequisite for us to release Iceberg is to have proper style guidelines that are enforced in continuous integration. I would like to propose adding code linting using an open-source linting toolkit called Baseline [github.com]. I have submitted a pull request [github.com] to integrate Baseline with the iceberg-api module. In that pull request, I describe some of the style conventions that were adopted that were not enforced before, as well as cases in which we deviate from Baseline’s style guidelines in favor of Iceberg’s prior opinions. Please look over the pull request and the proposed changes, and provide any feedback you may have on this mailing list thread or in pull request comments. Once we’re satisfied with the style rules we’re committed to holding to, I can submit patches to apply linting to all of the other modules, but would certainly appreciate help in working through these. Let me know if you would be interested in contributing to this effort as well. Thanks, -Matt Cheah -- Ryan Blue Software Engineer Netflix
smime.p7s
Description: S/MIME cryptographic signature
