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.

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

Update aliases to inherit from a single alias per site #176

Open
damienmckenna opened this issue Sep 17, 2014 · 2 comments
Open

Update aliases to inherit from a single alias per site #176

damienmckenna opened this issue Sep 17, 2014 · 2 comments
Labels

Comments

@damienmckenna
Copy link

Rather than duplicating lines on each alias, making it harder to improve the configuration, why not output one complete definition and then output others that inherit from the first one and only override the values necessary (uri, remote-host, remote-user)?

@mikevanwinkle
Copy link
Contributor

This is a good idea, but I suspect this would be a change that would need to take place in drush itself and not just the terminus plugin. Can you open a ticket there and close this one once you done so. Thanks again for the idea.

@joshkoenig
Copy link
Contributor

This is also not a Terminus question, but rather one for how we generate alias files. You can get said files through Terminus, but it's not making them, just getting them the same way you would in the UI.

That said, I think what @damienmckenna is getting at is clever. There's a lot of repetition in our generated aliases. Given that they are raw PHP, we could so something simpler by defining a skeleton and iterating over that with specific values.

This is worth considering as we think through supporting Drush7 and changing the logic of how we generate alias files generally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants