Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

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

Aberta
#95 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
58/100
Tipo de issue
Refatoração
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
github-actions, python
Domínio
ci-cd, testing

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
Python
Estrelas
0
Forks
0
Merge médio
35min
PRs com merge (30d)
43

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

  • Sem Dockerfile nem arquivo Docker Compose
  • Sem modelo de pull request
  • Sem guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de nyjc-computing/campus-api-python

Todas as issues de nyjc-computing/campus-api-python

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.