thebigmunch/audio-metadata

Add strict/lenient modes with toggles

Ouverte

#18 ouverte le 15 févr. 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Python (13 forks)github user discovery
designenhancementfeaturehelp wanted

Métriques du dépôt

Stars
 (60 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Change certain exceptions, to be determined, to warnings that can be made exceptions based on which mode audio-metadata is in. The exceptions most likely to be changed regard spec-compliance of tags. This would be the precursor of the ability to fix/upgrade tags/tag versions.

  • Which should be default? Current behavior is strict.
  • What API to use to toggle mode?
    • Top-level API functions to enable globally?
    • Arguments to top-level API functions?
    • Both?
  • How will the warnings classes fit in with exceptions hierarchy [#17]?

Guide contributeur

Direction de recherche
Examinez la hiérarchie actuelle des exceptions dans audio metadata, comprenez quelles exceptions sont liées à la conformité aux spécifications. Concevez un mécanisme de bascule (drapeau global, argument par fonction, ou les deux) pour passer entre les modes strict et permissif. Considérez comment les avertissements seront représentés et comment ils s'intègrent dans la hiérarchie des exceptions existante. Regardez des bibliothèques similaires (par exemple, mutagen) pour des modèles.
Stack technique
python
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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.
Plutôt claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50