getappmap / getappmap/appmap-python

Allow creation of Config instances

Ouverte
#33 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Python
Étoiles
104
Forks
17
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Not sure about making it a singleton -- singleton configuration tends to come back biting me later. (I can imagine, for example, when trying to drive, from a python script, a more complicated project with parts needing different settings.) Would it be a good idea to instead have a default instance but allow constructing and using others? Could also make it easier to test if it becomes more complicated.

Originally posted by @dividedmind in https://github.com/applandinc/appmap-python/pull/31#discussion_r575681654

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Cette issue est une ancienne discussion de conception et ne nomme aucun fichier, test ou point d’entrée. Commencez par localiser l’implémentation actuelle de Config et l’utilisation de son instance par défaut, puis déterminez si des configurations construites indépendamment peuvent coexister sans casser les appelants existants. Le travail est terminé lorsque la conception est arrêtée, que plusieurs paramètres peuvent être utilisés en toute sécurité et que le comportement est couvert par des tests.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
tooling
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.