Skip to content

Conversation

@f213
Copy link
Member

@f213 f213 commented Aug 27, 2025

По опыту школы делает тесты гораздо более читаемыми, сайдэффектов не вызывает

@f213 f213 requested a review from kazqvaizer August 27, 2025 05:08
@kazqvaizer
Copy link
Contributor

Строго говоря в этой реализации есть сайд эффект, к примеру, поле modified, которое завязано на базовый save и которое мы часто используем - не сохранится. В целом семантика приближает нас к update методу в QuerySet, который работет в этом смысле также (т.е. не вызывает save). В этом смысле я не против.

@f213 f213 merged commit 7bfb4cf into master Sep 1, 2025
3 checks passed
@f213 f213 deleted the f213-patch-1 branch September 1, 2025 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants