-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
chore: cleanup and rearrange unclassified configs into feature config groups #7586
Conversation
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.
LGTM
Hi @bowenliang123. This would be a great opportunity to apply the Ruff Formatter within the config/ directory. Would you be interested in implementing this change? |
Sure. That goes with #7590 for the configs, #7591 for the tests and #7592 for the db migrations. |
Awesome! Thank you for doing these! |
Checklist:
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint godsDescription
@computed_field
for display messages, which is week in reliability and misuse of configs:Type of Change
Testing Instructions
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration