Closed bproffitt closed 3 years ago
I'm merging this PR as the changes are not substantive but more formatting and style. Since the author is on leave for the time being, I'm going to be a bold editor and move this forward.
Thanks for this everyone. Although it has already been merged I completely agree with this review and will create a separate PR to the action points raised :+1:
Thanks for this everyone. Although it has already been merged I completely agree with this review and will create a separate PR to the action points raised +1
Thank you! And thanks for understanding my merging your PR before your review, I was getting confused in hat-wearing (author? editor? take charge and get it done-er?) and also wanted to move things along.
I will need to do a copyedit and/or style review to make sure that things align with the rest of the general formatting we're doing; which I'll set up as a non-blocking pull request.
Thank you! And thanks for understanding my merging your PR before your review, I was getting confused in hat-wearing (author? editor? take charge and get it done-er?) and also wanted to move things along.
I think that this is a good example of an acceptable situation when an author goes AWOL (even if temporarily) and how we can retrospectively have alterations to reviews when or if the author returns. An editor of the core team should have the authority to merge format and non substantial changes IMO for the benefit of the project. Utilising the "agile" methodology in the editorial process :smile:
Development edit of chapter, #70