Skip to content
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

Auto-discovery of multisite.name #1087

Closed
danepowell opened this issue Feb 14, 2017 · 0 comments
Closed

Auto-discovery of multisite.name #1087

danepowell opened this issue Feb 14, 2017 · 0 comments
Labels
Enhancement A feature or feature request

Comments

@danepowell
Copy link
Contributor

Currently, multisite projects must manually override the multisite.name array to add the name of each multisite: https://github.com/acquia/blt/blob/8.x/phing/build.yml#L87

It seems like it would be pretty easy to auto-discover sites in the sites directory and build this array automatically, making for an easier and less error-prone setup process.
@grasmash grasmash added the Enhancement A feature or feature request label Feb 14, 2017
@danepowell danepowell mentioned this issue Feb 17, 2017
Closed
@bobbygryzynger bobbygryzynger mentioned this issue Feb 21, 2017
Merged
grasmash pushed a commit that referenced this issue Feb 22, 2017
@bobbygryzynger @grasmash
Fixes #1087: Auto-discovery of multisite.name. (#1119)
a220e86 
* Auto-populate multisite.name if not provided by user.

* Add testing for multisite properties.

* Return an empty string if property doesn't exist.

* Update expected multisite.name value, and make configurable.
@danepowell danepowell mentioned this issue Feb 22, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement A feature or feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@grasmash @danepowell