<p dir="ltr">Hi folks, I've recently had a few questions about the FBX community's merge policy, and wanted to bring them to the list, because it might be helpful to have a well documented standard available on the wiki.</p>
<p dir="ltr">I would like to hold myself to something like the following:</p>
<p dir="ltr">- Unit tests pass.<br>
- Style tests (pep8.py, etc.) pass.</p>
<p dir="ltr">This at least makes sure that code hasn't become broken and that it looks similar to other code already in the project.  This isn't something I've enforced (FreedomBuddy doesn't *have* any tests other than acceptance tests), but something I'd like to start sticking to, in order to set clear expectations for patchers.</p>
<p dir="ltr">What other criteria do other folks use?</p>
<p dir="ltr">Thanks,<br>
Nick</p>