Commits (41)
-
20th (Victor Nikulshin) authored
If function 'deploy' is defined in __init__.py it masks the imported module with the same name. Placing it into deploy.py file is more logical structure-wise. To make sure that the command is still accessible by its old name, it is marked as a default task for 'deploy' module.
aeb7befe -
20th (Victor Nikulshin) authored7bf32a18
-
20th (Victor Nikulshin) authored
Adding and removing blank lines where appropriate to be more in accordance with Python coding standards.
93209c69 -
20th (Victor Nikulshin) authored13472b82
-
20th (Victor Nikulshin) authored
Exposing tasks from deploy module by their old names in addition to namespaced names for backwards compatibility.
35149929 -
20th (Victor Nikulshin) authoredba7fc7fa
-
20th (Victor Nikulshin) authoreddd737731
-
Victor Nikulshin authored
Reformat code to match Python coding standards See merge request !17
03eddeb6 -
b37f3851
-
If configured host name is empty, Drupalizer will construct an invalid `sed` invocation to modify `/etc/hosts` file. As the result, environment initialization will fail.
b4104eac -
It is better to use the most common IP address assigned to Docker containers than to use a network broadcast address.
daeb29d5 -
Victor Nikulshin authored
Change default vars values to something that does not crash Drupalizer See merge request !20
b2e327dc -
b5b2080e
-
Victor Nikulshin authored
Do not update /etc/hosts file if target hostname is not set See merge request !21
d4aa5137 -
Do not try to download translation if no languages were set in site configuration. Otherwise, Drush will try to download translations, will be able to find anything, and ultimately fail environment initialization.
12720aef -
Locale configuration during installation is done with two config parameters: `locale` and `site_languages`, which is excessive. Given that the `locale` parameter is a simple boolean, its value can be inferred from the value of `site_languages` parameter.
c5f4687e -
The `site_languages` parameter can receive multiple language codes separated by commas. Thus its value cannot be reused as-is as a value of `locale` parameter which is passed to Drush during site installation. The value of `site_languages` must be split, and the first element in the list of languages will be used as a site locale during installation.
1e1c056f -
Victor Nikulshin authored
Regularize usage of locale and site_languages config On n'a pas de besoin d'avoir deux parameters qui configure la locale d'installation. Seulement site_languages is suffisant. See merge request !23
669cfc07 -
The `fab_run()` function does not add any behavior for local calls, only ambiguity of the command execution environment. Inlining `local()` calls contributes to a clear separation of enrivonments.
8ee7032f -
It makes sense (and even possible at all) to controll Docker containers from local environment. Replacing `fab_run()` directly with `local()` calls ensures that container management will not be performed inside of Docker instance or on remote server because of a configuration error.
08884967 -
This argument is valid only in combination with `local` role parameter, and it is not being called in this combination anymore.
ca0a0c05 -
The `provision` task specifically targets Jenkins platform that can only exist in local environment. Direct usage of `local()` operation prevents accidental manipulation of build artifacts in other environments.
9e4c6bf1 -
adb5acd7
-
Victor Nikulshin authored
Do not use `fab_run()` helper for local tasks Le `fab_run()` helper n'ajoute aucun fonctionnalité à l'exécution des commandes locales, mais il fair plus difficile à comprendre où un commande sera exécuter. Si on minimalise utilisation de `fab_run()`, le workflow de Drupalizer deviens plus transparent. See merge request !22
f8fee175 -
Samuel Sirois authored722a8c88
-
Samuel Sirois authored3ee7da9f
-
Emmanuel Milou authored19d47c0d
-
Victor Nikulshin authored
Feature/docker.clean Removes container and images for a fresh workspace See merge request !24
c6acd3e4 -
Victor Nikulshin authored749e0863
-
Victor Nikulshin authored
Use fabric API to execute subtasks in docker.clean See merge request !25
4df2c024 -
Victor Nikulshin authoreddc08a1bc
-
Victor Nikulshin authored
Import missing execute() from fabric api into deploy.py See merge request !26
17991494 -
Victor Nikulshin authoreddb860fc5
-
Victor Nikulshin authored
Import missing lcd() from fabric api into deploy.py See merge request !27
15215b0d -
Philippe Mouchel authored64944347
-
Emmanuel Milou authored
Use of always_use_pty variable
e8b41917 -
Victor Nikulshin authored
Exclude `files/` directories and `settings.php` files from rsyncing.
162cf746 -
Victor Nikulshin authored
Rewrite rsync exclude rules in deploy task Exclude `files/` directories and `settings.php` files from rsyncing. See merge request !28
cabe8c77 -
Philippe Mouchel authored4e000231
-
Samuel Sirois authored
We should parametrize this option through drush's make/yml files. This will avoid an error that occurs in some situations where the folder to specified in contrib-destination gets overwritten by a git clone. e.g. : profiles/<my-profile> gets overwritten by a git clone while it might have been parent folder of contrib modules previously downloaded. This depends on the order a drush make/yml file has been written as far as I can understand the issue.
dc8702f4 -
Victor Nikulshin authored
Remove drush option <contrib-destination> from "prepare install" command We should parametrize this option through drush's make/yml files. This will avoid an error that occurs in some situations where the folder to specified in contrib-destination gets overwritten by a git clone. e.g. : profiles/<my-profile> gets overwritten by a git clone while it might have been parent folder of contrib modules previously downloaded. This depends on the order a drush make/yml file has been written as far as I can understand the issue. See merge request !29
25fd4ae6