drutiny / content

Content repository for open source Drutiny policies and profiles.
GNU General Public License v3.0
2 stars 9 forks source link

110 Incompatible Modules #8

Closed shanefjordan closed 6 years ago

shanefjordan commented 6 years ago

This is to resolve the Drutiny-cs-adaptor issue #110.

briantully commented 6 years ago

Awesome work, Shane! These new policies will be extremely helpful :)

In terms of process, do we need to squash recaptcha (79733dc & 2362c85) and varnish (652b2c5 & fdc8164)?

shanefjordan commented 6 years ago

I've cleaned up the policies that would be more dependent on Acquia hosting. I've also added the depend statements to the policies to keep reporting clean.

shanefjordan commented 6 years ago

@fiasco The checks that are related to Acquia Cloud, should I move them to the cs-adaptor project? If so, I believe I'll have to separate them into subfolders for drupal-8 and drupal-7 since the names are the same.

I wanted to check before making the changes and having to move everything again.

fiasco commented 6 years ago

No, there are acquia policies in this repository, they just need to have Acquia in the name of them since they are realm specific.

I think there is also an Acquia folder that they should be placed into. That folder can have drupal7/8 subfolders too.

shanefjordan commented 6 years ago

@fiasco Everything should be sorted into the proper locations now and ready for review.

fiasco commented 6 years ago

Thanks Shane, this is awesome!