Per @joestewart at #3130712: Test and apply new version of the patch we need for Security Review for Drush 9, we can try to use ^1.0 for the version of Security Review to install.

After your review the patch was committed. #2912708-17: Update drush commands to work for 9.x version of drush

It might be possible to use "drupal/security_review:^1.0" now as well ( this would still require the patch).

Comments

shrop created an issue. See original summary.

  • shrop committed b59cefa on 8.x-1.x 
    Issue #3132755 by shrop, joestewart: Update Security Review to ^1.0

shrop credited joestewart.

shrop’s picture

shrop commented
Status: Active » Fixed

Thanks to joestewart for mentioning this. Here is what I did:

  • Compared the 1.0-alpha1 tag against the latest 8.x-1.x branch to ensure they are close enough to try to apply our existing patch with 1.0-alpha1
  • Added ^1.0 to composer.json for security_review
  • Committed change
  • Ran a Guardr Project build to ensure everything passes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.