getappmap / getappmap/appmap-python
Allow creation of Config instances
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
- 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.
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