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 <https://github.com/palantir/gradle-baseline>. > > > > I have submitted a pull request > <https://github.com/apache/incubator-iceberg/pull/143> 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
