Update sites.yml - #20
Conversation
|
These should probably both be opt-in options rather than defaults. |
|
@otakup0pe I think that's exactly what this does? @wgregorian would you mind also updating the documentation/readme and defaults (should be empty hashes) to reflect this change in the PR? |
|
@pjan I was reviewing my changes from 5 months ago, I believe the defaults are already in place in README for this PR since it re-uses all existing vars. Have I missed something really obvious? |
|
I've added a similar change for myself but introduced a variable |
|
I would also prefer the distinction between enabled and available sites to be represented by different variables. |
|
Ability do enable/disable sites was introduced with #27. |
|
closing this in favour of #29 |
It made sense to create the directories and enable the virtual sites for me. Wondering if you think it's a good idea?