-
Notifications
You must be signed in to change notification settings - Fork 23
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
Make the update helper command easier to run #108
Comments
I would like opinion about what is commented in the Pending tasks section. In my opinion I think a) is okay but I would like to know if it is okay for everyone. About the b) solution, I think it will lead to adding the developer author to all the updates done and that leads to:
|
IMO the update command should use a default git user if the param is not set or empty, something generic like "script@drupal-update" (thinking in the new name of the repository). I think this is key, as the script can be run without the boilerplate The make file will have this declared but empty, and the user can change it manually or as part of the composer install. |
There are two situations where this command is run:
On case 1) I think commits should have the dev's Git info because that dev is directly responsible of the commits. We can do this:
I'm not sure were to save that info. The Makefile seems like a bad place because it a file with commands. It should read the conf from another place. The .env file is an option, but that's relates to and environment, not the whole project. About getting the Git info to store, I guess it is ok to ask it on the create project run. |
I agree on the default fallback when there's no git info available. |
AFter a talk we have decided to not add the author. The update helper script will add a default author if it is not set. Thanks for the proposals. |
Problem description
Currently, the update helper command needs a list of parameters that are always repeated: the author and the environment list (drush aliases of the local sites). The main problem is that the environment list may become pretty large and missing one environment leads to the configuration of that site not being correctly consolidated. Also, the author's parameter syntax may become tricky and can lead to human errors.
Proposed solution
Create two make commands:
The author and the environment will be configurable in the Makefile. The environment list must be the list of drush aliases, each alias is a specific site of the project.
The author is the git commit author of the updates, it will be always the same. It must be decided by the team thinking that the author should be a git user that will never be removed from the project. So it should be a development account that is not related to any developer that may leave the project.
Pending tasks
How to set the author in composer create-project installations. if it is not set, the make update command won't work the first time.
Proposed solutions:
a) Ask for the git author in the composer create-project execution.
b) Set the git author in the composer create-project execution with the current git author information of the local user.
The text was updated successfully, but these errors were encountered: