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

Django 5.1 ja Python 3.12 #65

Merged
merged 39 commits into from
Dec 2, 2024
Merged

Django 5.1 ja Python 3.12 #65

merged 39 commits into from
Dec 2, 2024

Conversation

ilesoft
Copy link
Contributor

@ilesoft ilesoft commented Nov 26, 2024

Tässä on vaiheittain ja vuorotellen koroteltu Django ja Pythonin versiota. Jokainen committi on koitettu testata, ja mahdollisuuksien mukaan seuraavaa versiota on lähdetty korottamaan vasta kun testit on saatu ajettua -We-lipulla.

Tämä tullaan mergeämään vasta kun tuo PR #55 saadaan alta, muttta katselmoinnin voi jo tehdä. Tehty

@ilesoft ilesoft requested a review from ZeiP November 26, 2024 14:30
@pholm
Copy link
Contributor

pholm commented Nov 29, 2024

Yleisenä huomiona, välttäisin eri ohjelmistoversioiden (esim. Django, Python, black conffit) mainitsemista dokumentaatiossa. Nyt vaatii kehittäjältä aina muistamista mihin kaikkialle tieto pitäisi päivittää. requirements.txt / muut conffitiedostot ovat nähdäkseni riittävä "single source of truth".

@ilesoft
Copy link
Contributor Author

ilesoft commented Dec 1, 2024

Yleisenä huomiona, välttäisin eri ohjelmistoversioiden (esim. Django, Python, black conffit) mainitsemista dokumentaatiossa. Nyt vaatii kehittäjältä aina muistamista mihin kaikkialle tieto pitäisi päivittää. requirements.txt / muut conffitiedostot ovat nähdäkseni riittävä "single source of truth".

Olen samaa mieltä. Mutta kun noi kirjastoversiot nyt oli tuolla dokumentaatiossa niin en lähtenyt muokkaamaan dokumentaation muotoa tässä PR:ssä. Seuraavassa PR:ssä noi versiot poistuu tuolta dokumentaatiosta.

Base automatically changed from python3-3 to develop December 2, 2024 13:17
Using shorthand "__all__" is counted as
explicit here.

Remove unused inlineformset_factory and ModelForm.

These fixes are due to deprecation in Django 1.6
and future removal in Django 1.8.
This fix is due to future change in functionality in Django 1.8.
Remove souht as Django 1.7 has built-in migrations.
In Django 1.10, session verification will be enabled regardless
of whether or not SessionAuthenticationMiddleware is enabled.

This is made to silence deprecation warning.
 django.conf.urls.patterns is deprecated
 and will be removed in Django 2.0.
This set value tells to load templates from app's
subdirectory named "templates"
Add needed context processors.

Remove redeclaration of default template loaders.
on_delete will become a required argument in Django 2.0.
Error was:
Cannot override the namespace for a dynamic module that provides a namespace.
Old-style middleware using settings.MIDDLEWARE_CLASSES is deprecated.
SessionAuthenticationMiddlewar was removed as planned in commit
"Add django.contrib.auth.middleware.SessionAuthenticationMiddleware".
* Square brackets in character classes are escaped.
* Raw string is used for regexes.
* Square brackets not escaped when not a part of character class.
* Match parenthesis with character class
Also ifnotequal tag with if and "!="
Django's default automatic primary key is changed from
integer to 64 bit integer. Define that DEFAULT_AUTO_FIELD
so that old database doesn't cause warnings on startup.
This is the default, but it is going to change in
Django 5.0 to be enabled by default.
Add CSS class to make sure that formsets are
not spread to multiple "lines".
Copy link
Contributor

@pholm pholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oikein hyvältä näyttää. Jälleen kommentteja setuppiin liittyen, mutta voi ignorata tämän osalta

README.md Show resolved Hide resolved
@ilesoft ilesoft merged commit 161398d into develop Dec 2, 2024
3 checks passed
@ilesoft ilesoft deleted the django5-1 branch December 2, 2024 19:26
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.

2 participants