Support both Drupal 8 and Drupal 7 in the same version of site_audit

Hi Mark, I don't have a practical solution off the top of my head. I'm going to experiment a bit, you might be able to create a drush folder in the site root and that might take precedence, but that doesn't work on a platform unless you made a symbolic link in every single site back to the correct version of site_audit. That's suboptimal.

It's probably going to be best if the 8.3.x version supports both Drupal 7 and 8.

When I started the Drupal 8 version a couple years ago, there were drush compatibility issues that guided my decision. In retrospect, that was a bad call and now I need to merge them.

I would like to see this, too. I imagine it will be a ton of work.

In the mean time, can you at least mention on the project page that users should download the correct branch to work with their version of Drupal?

As a work-around, we could try something like this after downloading the git repository as ~/.drush/site_audit:

#!/bin/bash
# file drush_sa_7
git -C ~/.drush/site_audit checkout 7.x-1.x
drush "$@"

and then invoke, for example drush_sa_7 audit_all.

Make a similar file drush_sa_8. Both of them have to be somewhere in your $PATH.

Older versions of git do not support the -C option, so use --work-tree=$HOME/.drush/site_audit --git-dir=$HOME/.drush/site_audit/.git instead.

I guess that's going to be difficult on the long run. Drupal 9 is released now and requires Drush 10. While Drupal 7 requires Drush 8. In the meanwhile the project type also has changed to Drupal module in the current 8.x-3.x branch. I tend to close this as won't fix and focus on updating the docs to highlight the differences this makes now.

It makes sense to close this issue. I care much less about this issue now than I did in 2016.

👍