Install-DbaDarlingData - Missing Procedures and old domain in documentation about dbatools

[andreasjordan]

Thanks for the issue, I will take care of that today.

Looking at the code, I found a bug: $Procedure is never used, so we always install all procedures. I don't what to remove the parameter, so I will change the code to be able to install every procedure seperatly.

[andreasjordan]

Exactly.

[erikdarlingdata]

Cool, thanks. Perhaps a useful parameter input would be All, which would reference the DarlingData.sql file?

[andreasjordan]

But using DarlingData.sql has a drawback: With a list of individual procedures (and files), we output "Installed/Updated" per file and also check that we have SQL Server 2016 or higher for sp_QuickieStore.
With DarlingData.sql we always output "Installed".

So: Should All use DarlingData.sql or just run all the individual scripts?

Will open a pull request with using DarlingData.sql as I've just coded and tested that. But I can change that later...

[erikdarlingdata]

@andreasjordan whatever is easiest for you, really! I want to respect your time and volunteer effort, here.