entrouvert
/
hobo
16
0
Fork 0
Code Issues Pull Requests 16 Releases Activity

multitenant: pass --skip-checks to migrate command (#74968) #22

Merged
vdeniaud merged 1 commits from wip/74968-ajouter-skip-checks-sur-la-comma into main 2023-03-01 13:51:36 +01:00
Conversation 1 Commits 1 Files Changed 1 +1
vdeniaud commented 2023-03-01 12:07:32 +01:00
Owner
Copy Link
No description provided.
vdeniaud added 1 commit 2023-03-01 12:07:32 +01:00
fpeters reviewed 2023-03-01 12:08:52 +01:00
fpeters left a comment
Owner
Copy Link

J'imaginais en fait qu'on pouvait poser requires_system_checks = False sur la classe. (mais pas testé etc.).

J'imaginais en fait qu'on pouvait poser requires_system_checks = False sur la classe. (mais pas testé etc.).
vdeniaud commented 2023-03-01 12:26:45 +01:00
Author
Owner
Copy Link

Cette option est déjà mise et elle ne fait plus ce qu'elle dit pour la commande migrate depuis https://code.djangoproject.com/ticket/31055 (mergé dans django 3.1).

Le patch de ce ticket ajoute un appel à self.check sans vérifier self.requires_system_checks (mais en vérifiant skip_checks, d'où mon patch).

C'est toujours le cas dans main, visiblement c'est un comportement qui n'a dérangé personne...

Cette option est déjà mise et elle ne fait plus ce qu'elle dit pour la commande migrate depuis https://code.djangoproject.com/ticket/31055 (mergé dans django 3.1). Le patch de ce ticket ajoute un appel à `self.check` sans vérifier `self.requires_system_checks` (mais en vérifiant `skip_checks`, d'où mon patch). C'est toujours le cas dans main, visiblement c'est un comportement qui n'a dérangé personne...
fpeters approved these changes 2023-03-01 13:00:17 +01:00
fpeters left a comment
Owner
Copy Link

Ok go ainsi alors.

Ok go ainsi alors.
vdeniaud merged commit c766be2305 into main 2023-03-01 13:51:36 +01:00
vdeniaud deleted branch wip/74968-ajouter-skip-checks-sur-la-comma 2023-03-01 13:51:36 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/hobo#22
No description provided.
Delete Branch "wip/74968-ajouter-skip-checks-sur-la-comma"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?