drivendataorg/cloudpathlib

Missing dependency message does not appear when it should

Ouverte

#441 ouverte le 13 juin 2024

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Python (87 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (624 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Thanks for the great work on cloudpathlib! I noticed a bug in the documentation for using no_sign_request=True found here. When I attempt to use the example code, an error shows: NameError: name 'Session' is not defined. Here's an example reproducing the error: https://gist.github.com/d33bs/eb19f864baae89757e0e0969611cf1af

Guide contributeur

Direction de recherche
Trouvez où `Session` est utilisée dans le contexte de `no sign request=True` et ajoutez l'import manquant pour `boto3.Session` ou `requests.Session` selon le cas.
Stack technique
python
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85