Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1620 drobne porządki w importach #1630

Open
wants to merge 10 commits into
base: master-dev
Choose a base branch
from

Conversation

cptkidd62
Copy link
Collaborator

@cptkidd62 cptkidd62 commented Jan 3, 2024

Przejrzałam wszystkie pliki pod kątem innych importów zlokalizowanych niezgodnie regułami przy użyciu poleceń grep -r -A 10000 '^.*class ' | grep '.*import ', grep -r -A 10000 '^.*def ' | grep '.*import ', grep 'import ' $(grep -rL 'class') oraz grep 'import ' $(grep -rL 'def ') (uznając słowa kluczowe class i def za pewną granicę między importami z początku pliku a jego resztą) i nie wykazały one żadnych anomalii poza tymi wymienionymi w issue #1620. W plikach tych przeniosłam wszystkie importy na początek tychże plików, ponieważ nie znalazłam żadnego uzasadnienia, żeby musiały tam pozostać.

Dodatkowo, w pliku schedule/models/event.py zmieniłam zapytania Term.objects.filter(event=self) na self.term_set.all() w celu wyeliminowania importu Term w tym module.

Natomiast w aplikacji enrollment przeniosłam ChangedDay oraz Freeday do osobnego pliku specialdays.py, żeby uniknąć problemów z importem tych klas. Statyczne metody get_day_of_week i get_python_day_of_week z modułu Term dla porządku przeniosłam do pliku common/days_of_week.py, a metodę get_day_of_week z ChangedDay przemianowałam na get_official_day_of_week.

Na swoim miejscu pozostał import django.contrib.auth.models w pliku users/apps.py, który prawdopodobnie musi się znajdować wewnątrz funkcji ready.

Co do importów zakończonych komentarzem # noqa - wyszukałam wszystkie miejsca, w których występują i tam, gdzie nie było to dospecyfikowane, ograniczyłam ich działanie do reguły F401 (imported but unused), jako że jest to jedyna reguła, którą te linie łamią.

@cptkidd62 cptkidd62 linked an issue Jan 3, 2024 that may be closed by this pull request
@cptkidd62 cptkidd62 self-assigned this Jan 5, 2024
@cptkidd62 cptkidd62 requested a review from lgpawel January 5, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drobne porządki w importach
2 participants