Our contributor and committer guidelines haven't changed in a while. In light of the discussion that happened a few weeks ago, where high commit threshold was cited as one of the factors discouraging submissions, I suggest we discuss some ideas and see if the guidelines should be updated.
I have one. We pick some reasonable time period like a month after a PR is submitted. If the PR review process is still going on *and* there is a disagreement between the contributor and reviewer, we will look to see if the submission satisfies some acceptable criteria and if it does we accept it. We can discuss what those criteria should be in this thread. The basics should be met, such as code format, license, copyright, unit tests passing, functionality working, acceptable performance and resolving logical flaws identified during the review process. Beyond that, if there is a disagreement with code quality or refactor depth between committer and contributor or the contributor agrees but does not want to spend more time on it at that moment, we accept the submission and create a separate JIRA to track any future work. We can revisit the policy in future once code submissions have picked up and do what's appropriate at that time. Thanks
