Had a conversation with @abkfenris and we agreed it would be good to add a review requirement to pull requests before they can be merged in. Main reasons for this are to ensure that code being merged is not going to break something, but also to guarantee that we take the time to review each others work. You should notice next time you add a pull request that you will be unable to merge it until somebody has checked it.
Had a conversation with @abkfenris and we agreed it would be good to add a review requirement to pull requests before they can be merged in. Main reasons for this are to ensure that code being merged is not going to break something, but also to guarantee that we take the time to review each others work. You should notice next time you add a pull request that you will be unable to merge it until somebody has checked it.