drupal-with-nginx icon indicating copy to clipboard operation
drupal-with-nginx copied to clipboard

Published 20 hours ago • verified publisher icon perusio

Review request

Open iryston opened this issue 7 years ago • 3 comments

Hi, @perusio

Can you take a look at my fork I've made total synchronisation between branches, and publish them as D6-SYNC, D7-SYNC and master-SYNC. If you don't have objection for this work, I'll do pull request.

One serious change which can break working sites is 'Rename poll, status and ping'

All other changes are most cosmetic and typo fixes.

Regards, Igor

iryston avatar Aug 04 '16 01:08 iryston

Hi @iryston ,

I think you've done a great job. I would recommend you splitting your changes into separate independent features and create corresponding pull requests (PRs) to make code review easier.

kkomelin avatar Oct 24 '16 21:10 kkomelin

@kkomelin Thanks for recommendations. I'll recreate it in separate pull requests.

iryston avatar Oct 24 '16 21:10 iryston

Please recreate

andypost avatar Nov 09 '17 21:11 andypost