Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

chore: resolve 23 pyright diagnostics and enable a pyright CI job

Fermée
#95 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
58/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
Active
Stack technique
github-actions, python
Domaine
ci-cd, testing

Piste de recherche

Start by running poetry run pyright and inspect the diagnostics in campus_python/__init__.py, assignments.py, oauth.py, and the named test files. Compare the configuration and CI approach with the nyxchange-timetable-v2 reference cited in the issue, then review .github/workflows/python-tests.yml. Done when the 23 diagnostics are resolved or deliberately ignored and a pyright CI job runs successfully.

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

Description

chore

Observation

With pyright's default exclude list restored (chore/pyright-exclude, PR #94), running pyright 1.1.414 against the repo reports 23 errors, 0 warnings in project code:

  • campus_python/ sources: __init__.py ×6, assignments.py ×2, oauth.py ×1 (reportArgumentType / reportAttributeAccessIssue / reportOptionalMemberAccess / reportCallIssue / reportReturnType)
  • tests: test_unauthorized_refresh.py ×6, test_campus_request.py ×2, test_audit.py ×1, plus 1 reportMissingImports (a test-side sys.path import, presumably)

Ask

Resolve the 23 diagnostics (or scope the config with per-file ignores where deliberate), then add a pyright job to .github/workflows/python-tests.yml following the nyxchange-timetable-v2 pattern (poetry run pyright with pyright as a dev dependency). This repo is the client every student app pins, so it should be the first place the gate lands.

Context

Found 2026-10-04 while applying the same exclude fix as nyxchange-timetable-v2#64 (PR #94). This repo currently gates on pytest only; the nyxchange fix (PR there: #66) is the reference for config + gate.

Langage dominant
Python
Étoiles
0
Forks
0
Merge moyen
34 min
PR mergées (30 j)
44

Préparer son environnement

Ouvrir dans Codespaces

Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.

  • Aucun Dockerfile ni fichier Docker Compose
  • Aucun modèle de pull request
  • Aucun 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.

Autres issues de nyjc-computing/campus-api-python

Toutes les issues de nyjc-computing/campus-api-python

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

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