EasyEngine/easyengine

`EE_PROXY_TYPE` to be replaced by a utiliy function

Open

#1 388 ouverte le 6 févr. 2019

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)PHP (429 forks)batch import
good first issuekind/enhancement

Métriques du dépôt

Stars
 (1 984 stars)
Métriques de merge PR
 (Merge moyen 20j 14h) (7 PRs mergées en 30 j)

Description

Currently, EE_PROXY_TYPE is a constant, which is the container id of the global nginx-proxy container. It is computed every time during the initialization of an command as it is present in the runner.

The command executed for it is time consuming and also required by only a handful of commands, hence it should not be in the runner. Instead it can be replaced by a utility function which computes it's value only when required.

Once the value is found for the first time, it can be stored in a constant, and on next call is_defined function can be used in the function to check if the value is already there, to skip the time consuming computation for the second time.

Guide contributeur

Direction de recherche
Identifiez tous les endroits dans la base de code où `EE PROXY TYPE` est utilisé, puis créez une fonction qui calcule et met en cache la valeur, et remplacez toutes les utilisations.
Stack technique
php
Domaine
cli
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPHP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75