[Request/Question] Allow control over if/how the $env var is exposed
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 30/100
Piste de recherche
Commencez par retracer la manière dont la variable $env est exposée aux programmes expr et comparez ce chemin avec la directive existante DisableBuiltin(). Confirmez l’API prévue pour renommer ou désactiver $env, puis ajoutez une couverture montrant le comportement sélectionné et vérifiez que le comportement par défaut existant reste intact.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I'm considering using expr at DAYJOB and would like to not expose the $env var using that name. We're planning on using expr as a DSL and the domain has vars which conflict with $env and/or look similar enough that I want to stray from any confusion.
I skimmed thru the code and it doesn't look like I can hide it from the expr programs via configuration today. Please call out if I just missed that.
Questions I have for you:
- Does this var need to be exposed to end users? I can understand the convenience of it, but do you see it as a must have or a nice to have?
- Would you be amenable to allowing it to be exposed under a different name? If not, would you be amenable to allowing it to not be exposed (something like the DisableBuiltin() directive)?
I'm happy to do the work to support my request but I wanted to ask before I went and put up a PR that doesn't align with your direction for expr.
- Langage dominant
- Go
- Étoiles
- 8k
- Forks
- 535
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de expr-lang/expr
-
Docs: Built in functions shadow user variablesPeut-être pris @sanmaxdev l’a pris il y a 97 jours. Ouvertedocs needed
Difficulté 1/5 1-3 heures Accessibilité débutants 62/100
-
`in` operator on a struct doesn't respect `expr` tagsPeut-être pris @akashchamp l’a pris il y a 17 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
-
`find`, `findIndex`, `first` and `get` are type-checked as the element type where they return nilPeut-être pris @TastyHeadphones l’a pris il y a 13 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
-
Release latest master branchOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
Toutes les issues de expr-lang/expr
Issues similaires
-
bug frontend good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
oalders/clodhopper#133 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
peasant-labs/peasant#596 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
hatchet-dev/hatchet#5179 ·
Les mainteneurs répondent en général sous 1 jour
-
bug triage
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
FairwindsOps/nova#484 ·