-
-
Notifications
You must be signed in to change notification settings - Fork 31.1k
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
Fixed #32582 -- Removed unnecessary dot in names of cloned test databases on SQLite. #14169
Conversation
Hello @girishsontakke! Thank you for your contribution 💪 As it's your first contribution be sure to check out the patch review checklist. If you're fixing a ticket from Trac make sure to set the "Has patch" flag and include a link to this PR in the ticket! If you have any design or process questions then you can ask in the Django forum. Welcome aboard ⛵️! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@girishsontakke Thanks for this patch 👍
Please add tests to the tests/backends/sqlite/test_creation.py
.
Thank you @felixxm for your review and suggestion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@girishsontakke Thanks for updates 👍 I left comments.
Co-authored-by: Mariusz Felisiak <felisiak.mariusz@gmail.com>
Co-authored-by: Mariusz Felisiak <felisiak.mariusz@gmail.com>
@girishsontakke Welcome aboard ⛵ |
Fixed https://code.djangoproject.com/ticket/32582.
I am not sure in which section of the project I would have to write a test for this patch but if someone guides me then I will definitely write a test for this patch.