-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Static create commands #5562
Static create commands #5562
Conversation
I wasn't sure why the Sql command tests were failing after the conversion, so I reverted that part. The generators could take greater advantage of DI now, but overall I think it would be good to merge here now, move on to other things, and maybe come back to those issues later, as follow-on work. |
I just remembered I was going to use |
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.
Looks great. Feel free to merge when ready.
…ds, RoleCommands and WatchdogCommands.
…tter at all if the service is missing; will make a wrapper for this as follow-on work.
This reverts commit 5c2281d.
41059c9
to
bb8e240
Compare
Continuation of #5552, rebased on top of #5553