New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

'all' is not a valid multisite #4362

Merged

danepowell merged 1 commit into acquia:main from danepowell:DX-1723

Mar 10, 2021

Contributor

danepowell commented Mar 10, 2021 •

edited

Motivation

BLT considers "all" to be a valid multisite, which it's obviously not.

Proposed changes

Exclude 'all' from the list of returned sites.

Merge requirements

Major change, Minor change, Bug, Enhancement, and/or Chore label applied
Manual testing by a reviewer


          DX-1723: 'all' is not a valid multisite

78b13fc

danepowell added the Bug label

danepowell merged commit 1053440 into acquia:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment