accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dave Marion <dlmar...@comcast.net>
Subject [DISCUSS] Pull Request Guidelines
Date Mon, 05 Jun 2017 15:08:29 GMT
I propose that we define a set of guidelines to use when reviewing pull requests. In doing
so, contributors will be able to determine potential issues in their code possibly reducing
the number of changes that occur before acceptance. Here's an example to start the discussion:


Items a reviewer should look for:

1. Adherence to code formatting rules (link to formatting rules)

2. Unit tests required

3. Threading issues

4. Performance implications


Items that should not block acceptance:

1. Stylistic changes that have no performance benefit

2. Addition of features outside the scope of the ticket (moving the goal post, discussion
should lead to ticket creation)

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message