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

Fixed #35429 -- Added argparse choices to --database options. #18130

Merged
merged 1 commit into from
May 10, 2024

Conversation

saJaeHyukc
Copy link
Contributor

@saJaeHyukc saJaeHyukc commented May 4, 2024

Trac ticket number

ticket-35429

Branch description

Provide a concise overview of the issue or rationale behind the proposed changes.

Checklist

  • This PR targets the main branch.
  • The commit message is written in past tense, mentions the ticket number, and ends with a period.
  • I have checked the "Has patch" ticket flag in the Trac system.
  • I have added or updated relevant tests.
  • I have added or updated relevant docs, including release notes if applicable.
  • I have attached screenshots in both light and dark modes for any UI changes.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello! 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 ⛵️!

@saJaeHyukc saJaeHyukc force-pushed the issue-35429 branch 3 times, most recently from 1d5c9b7 to a7124ec Compare May 4, 2024 07:11
@adamchainz
Copy link
Sponsor Member

Thanks for the change and test, good work getting this started. The point of the ticket is to edit all relevant commands not just migrate - please track down every one that has a --database option.

Copy link
Sponsor Member

@adamchainz adamchainz left a comment

Choose a reason for hiding this comment

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

All commands

@saJaeHyukc
Copy link
Contributor Author

saJaeHyukc commented May 4, 2024

All commands

Thank you :) I will re-apply it after all the changes

@saJaeHyukc saJaeHyukc force-pushed the issue-35429 branch 5 times, most recently from 9bdbbd8 to 4fcc7d9 Compare May 4, 2024 13:57
@saJaeHyukc saJaeHyukc force-pushed the issue-35429 branch 2 times, most recently from f4bcc25 to 021f7ae Compare May 8, 2024 12:24
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

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

This looks good thank you @saJaeHyukc 👍

@sarahboyce sarahboyce merged commit 4a76ac0 into django:main May 10, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants